-
Notifications
You must be signed in to change notification settings - Fork 50
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
Nova parameters & decider parameters and proofs serialization #151
Conversation
I needed to modify the import of your |
ee47397
to
fae1be8
Compare
…_params and public inputs
fae1be8
to
b036124
Compare
dce7646
to
cbf02ec
Compare
This PR is now ready for review. |
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.
Some nits. The rest looks good.
cbf02ec
to
0e400e9
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!
Adds serialization for
Add also tests for it, which serve as examples of usage.
The idea is that with this PR, the full flow with Nova can be done with Prover's getting the serialized trusted setup parameters, and the Verifier getting the serialized proofs, inputs, and parameters, like they were from different instances/machines. An example of serializing and deserializing & verifying the proof (from the deserialized data) can be found at folding/nova/decider_eth.rs#L421 .
resolves #153