-
Notifications
You must be signed in to change notification settings - Fork 129
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
Adds page pseudorandom number generation algorithm #963
Conversation
@davecgh, comments addressed. Suggested text lgtm. |
@@ -0,0 +1,8 @@ | |||
# <img class="dcr-icon" src="/img/dcr-icons/QuestionTicket.svg" /> Ticket Selection Algorithm | |||
|
|||
Every block, 5 tickets are pseudorandomly selected from the ticket pool to [vote](../../proof-of-stake/overview/). Below is a high-level description of the ticket selection algorithm. A more detailed, low-level walkthrough of the ticket selection code can be found in this [video](https://www.youtube.com/watch?v=eysGWVhDFWY). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
link should be to a .md
file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. FYI, I do remember trying .md
before and it wasn't working, so went with the /
. I think my mistake was that with the /
, I was able to use a relative path that wasn't technically correct (../../proof-of-stake/overview/
) and it worked. Using .md
with that path doesn't work. If you use the correct relative path (../proof-of-stake/overview.md
) it does.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly its a relative path from the current source file to target source file. This looks good now.
This PR adds a page under /research (open to placing it elsewhere) describing the details of the algorithm used to pseudorandomly select tickets for voting ( Closes #941 ).
@davecgh, I'm pretty much just rewording your original reddit post for flow. Do let me know if I've somehow mangled it. Also, if there is any other relevant info you can think of (e.g. reasoning for why this algo was chosen, references, etc.), can add.