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

Once arkworks-rs/r1cs-std#129 ready, update nova+cyclefold circuit impl to use it #60

Open
arnaucube opened this issue Jan 11, 2024 · 0 comments

Comments

@arnaucube
Copy link
Collaborator

Currently in the Nova+CycleFold circuits there are some parts that take NonNativeFieldVar representations and hash them or check their values, but without doing actual computations.
The current ark-r1cs-std version automatically computes the limbs & bits needed for the NonNativeFieldVar in order to be able to operate with them (ie. sum, mul, etc).
In the described use case we don't need to operate with them but just to hash them or check their values, so once arkworks-rs/r1cs-std#129 is merged, we could make use of the custom configuration to reduce the number of limbs used, and therefore reducing the number of constraints taken by the AugmentedFCircuit of Nova.

Note: two weeks ago the NativeFieldVar was renamed to EmulatedFpVar: arkworks-rs/r1cs-std@ed2d55e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant