-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: Forward ports #150
feat: Forward ports #150
Conversation
f85fc30
to
895d8b3
Compare
78bd0cc
to
e036453
Compare
@huitseeker: Should I overwrite the S3 circuit artifacts for EDIT: I've finished reviewing the changes, everything looks good! I'm doing some more full plonk recursion tests to double-check the argumentcomputer/zk-light-clients#161 changes before I approve |
chore: format PR #934 (#939)
…te/prove/verify (#940) - CLI test now fails because of the breaking changes - Refactor a lot of the parallel three-type proof code in SDK to use the new `SP1Proof` enum.
Co-authored-by: Tej Qu Nair <[email protected]> Co-authored-by: John Guibas <[email protected]> Co-authored-by: Matt Stam <[email protected]>
Hook results must be accessed immediately after calling. One no longer has to rely on the `input_stream` being empty when calling a hook and accessing its results.
e036453
to
229d6c6
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.
I'm overwriting the v1.0.8-testnet
artifacts with the ones from this PR, and everything is good to go!
* feat: Update contracts to v1.0.8-testnet Companion to argumentcomputer/sphinx#150 * fix: Remove VKEY_HASH * chore: forge fmt --------- Co-authored-by: wwared <[email protected]>
This ports the following upstream PRs and pushes us halfway to v.1.0.0-rc.1:
ProverClient
to use builder pattern for execute/prove succinctlabs/sp1#940Note
This will need companion PRs.
Light client PR: argumentcomputer/zk-light-clients#161
Loam PR: https://github.com/argumentcomputer/loam/pull/214
Sphinx-contracts PR: argumentcomputer/sphinx-contracts#2