Skip to content

Update masternode.cpp#7

Open
dkrym wants to merge 1 commit intoCropdev:masterfrom
dkrym:patch-1
Open

Update masternode.cpp#7
dkrym wants to merge 1 commit intoCropdev:masterfrom
dkrym:patch-1

Conversation

@dkrym
Copy link
Copy Markdown

@dkrym dkrym commented Mar 12, 2018

Fix for correct last paid timestamp

Fix for correct last paid timestamp
@dkrym
Copy link
Copy Markdown
Author

dkrym commented Mar 12, 2018

Unfortunately I am not sure how to test it, I compiled it and it seems it shows correct time (or at least different time for nodes):
screenshot from 2018-03-12 11-21-29

But it seems it doesn't fix the winners list :(

@dkrym
Copy link
Copy Markdown
Author

dkrym commented Mar 12, 2018

I am sorry but are you sure it is as you wrote? I am new to this so not gonna argue but my understanding is winners is list of people who won so it shouldn't be used when selecting new winner. I thought you sort people by LastPaid and from top 10% you select someone. His LastPaid stamp is updated and he is moved to the end of queue.

Anyway thanks for your input, it is very useful to understand this.

Edit: Sorry you talking about "vWinning list" now I understand, cheers

@dkrym
Copy link
Copy Markdown
Author

dkrym commented Mar 12, 2018

I unfortunately don't have knowledge to create test net. Do you know what codebase is used? I am just wondering why Crop doesn't use new codebase where it is fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant