-
Notifications
You must be signed in to change notification settings - Fork 805
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
Keccak test cases #82
Conversation
4c4de4b
to
9ce3b78
Compare
01d9cdd
to
ae56d43
Compare
637f842
to
a4bfe62
Compare
a4bfe62
to
2219fce
Compare
43e22c6
to
aaf6d31
Compare
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.
LGTM
|
||
pub const PERMUTATION: usize = 24; | ||
|
||
pub static ROUND_CONSTANTS: [u64; PERMUTATION] = [ |
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.
Can we add a link to the place where we extracted the constants from?
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.
I've added links to where the constants are from.
Merged to unblock upcoming works. Feel free to give post merging review. |
Problem
We need some intermediate values between keccak steps to build test cases.
Solution
Modify https://github.com/Follow-DeFi/keccak-rust to get a version of arithmetic keccak.
TODO