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.

Sign up for GitHub

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

Jump to bottom

Correct primitive root #2754

Merged

djrtwo merged 1 commit into dev from dankrad-patch-1

Dec 1, 2021

Contributor

dankrad commented Nov 30, 2021

5 is not a primitive root of the BLS field, it needs to be 7


          Update beacon-chain.md

ad588d7

5 is not a primitive root of the BLS field, it needs to be 7

djrtwo approved these changes

View reviewed changes

djrtwo merged commit 4d35748 into dev

djrtwo deleted the dankrad-patch-1 branch

December 1, 2021 15:15

Member

ralexstokes commented Dec 1, 2021

for posterity's sake, @dankrad shared this script w/ me to validate this number:

https://gist.github.com/ralexstokes/f1d2b6557dd573ae496a752b7ab4e563

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

Labels

None yet