Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix incorrect number of blocks in a round #123

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

OlegStotsky
Copy link

Producers are actually producing 12 blocks per turn, not 6.

Producers are actually producing 12 blocks, not 6.
@OlegStotsky OlegStotsky changed the title Fix incorrect number of blocks in round Fix incorrect number of blocks in a round Jan 9, 2019
Copy link

@NiVisser NiVisser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is correct... Every BP produces for 6 seconds a total of 12 blocks , in a pool of 21 producers. This makes 256 blocks in rounds of 126 seconds.

@OlegStotsky
Copy link
Author

How is this incorrect? You basically wrote the same information that is in my PR.

@OlegStotsky
Copy link
Author

It's 252 blocks. Not seconds. I have no idea where you got the seconds idea from. You multiply 12 blocks by 21 producers. You get 252. 252 blocks per round.

@NiVisser
Copy link

NiVisser commented May 9, 2019

Sorry, you are right, I did a little brain fart. The seconds part came from the idea that whoever wrote this whitepaper, was refering to the 126 seconds and did actually mean seconds instead of blocks. Anyway, the orginal sentence confused me and that is why I started looking for answers, which you helped clarify. I hope they can commit the changes and get this issue resolved.

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.

2 participants