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

Section 3.1.2 of the document correction needed : there should be a || b(0,26) instead of || b(0,27) to add up to 32 bytes. #17

Open
ctoma73 opened this issue Oct 27, 2023 · 1 comment
Assignees

Comments

@ctoma73
Copy link

ctoma73 commented Oct 27, 2023

In Section 3.1.2 of the document it is stated, quote: The array has length 32 and contains the UTF-8 encoding of the string “v2.0.0” followed by 0x00- bytes, i.e. ver = 0x76322E302E30 ∥ b(0, 27).
I believe there should be a || b(0,26) instead of || b(0,27) to add up to 32 bytes.
I’ve attached the VSCode notebook file (it opens in VSCode and can be viewed), to execute and run all cells requires Python extension installed in VSCode. My Notebook was able to compute the HP correctly.
HP = 0x2B3B025E50E09C119CBA7E9448ACD1CABC9447EF39BF06327D81C665CDD86296.
hp_test.zip

Tasks

No tasks being tracked yet.
@cryptosidh
Copy link

Thank you! Yes, this is a typo and it should be b(0,26). This will be corrected in the next iteration of the specification.

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

No branches or pull requests

3 participants