-
Notifications
You must be signed in to change notification settings - Fork 946
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
Whisk: add preset files #3424
Whisk: add preset files #3424
Conversation
presets/minimal/whisk.yaml
Outdated
# Misc | ||
# --------------------------------------------------------------- | ||
# [customized] | ||
CURDLEPROOFS_N_BLINDERS: 2 |
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.
Curdleproofs always requires 4 blinders, because of how the ZK protocol is designed.
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.
Noting that https://github.com/nalinbhardwaj/curdleproofs.pie is able to validate a generated proof with those parameters
This looks good now! However it seems like merging this PR will cause a good amount of test breakage? |
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! Thanks!
* Add Whisk preset files * Use N=8 for minimal preset * Update spec_object var location
* Add Whisk preset files * Use N=8 for minimal preset * Update spec_object var location
Reduce shuffling sizes for minimal preset to have faster tests.
On offline discussions with @asn-d6 we noted that
CURDLEPROOFS_N_BLINDERS + WHISK_VALIDATORS_PER_SHUFFLE
must be a power of two, so for mainnet it's reduced from 128 to 124