-
Notifications
You must be signed in to change notification settings - Fork 124
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
Anonymous Components for Poseidon hashers #1169
Anonymous Components for Poseidon hashers #1169
Conversation
✅ Deploy Preview for maci-typedoc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
d1e1ea9
to
86674b3
Compare
86674b3
to
d2a312a
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.
@0xjei thanks, just left some code style comments.
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.
thanks for this!! much neater
e1d6ec3
into
privacy-scaling-explorations:refactor/circom-circuits
Description
This PR makes it possible to remove Poseidon Hasher related templates (e.g., Hasher3) by instantiating the
PoseidonHash()
template directly in the code. This reduces the number of lines of code and makes it easier to read.Related issue(s)
See privacy-scaling-explorations/zk-kit#131
Confirmation