-
Notifications
You must be signed in to change notification settings - Fork 401
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
rename keccak #6197
rename keccak #6197
Conversation
these are the two files that are the important - rest almost everthing is result of renaming. |
Why That seems more for what its used for (i.e. variable name) than what it is (i.e. class name) Pedersen is also a Hash; as is Poseidon; would generic I may have this wrong, but I think a Commitment scheme can use hashes, but a Cryptographic hash function is not necessarily part of a Commitment scheme? |
Yes, I think we can rename it to Hash insted of Commitment. |
+1 for |
or Hash32? |
I vote for Hash256. |
c93d35b
to
b587ed1
Compare
add keccak for computation fix test file rename commitment > hash256
2265936
to
cd98924
Compare
renamed to Hash256, ready for a review! |
With verkle tree, we would be using Pedersen in instead of Keccak for state related operations.
Also there are other chains, that we plan to support, using differnet commitment schemes (For eg. poseidon).
This PR replaces Keccak with a more generic class Commitment that can be used to represent any commitment scheme (32 bytes).
Changes
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
If yes, did you write tests?
Notes on testing
Current unittest already cover this properly
Documentation
Requires documentation update
Requires explanation in Release Notes