-
Notifications
You must be signed in to change notification settings - Fork 39
Add Root Prover #1193
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
base: master
Are you sure you want to change the base?
Add Root Prover #1193
Conversation
3f144d8 to
de74651
Compare
- Restructure the public values in recursion - Aggregate and check septic point summation in the aggregation tree
| // } | ||
| Ok(app_commit) | ||
| } | ||
| // pub fn verify_e2e_stark_proof( |
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.
This is commented out.
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.
This is the verifier for the last internal proof (in OpenVM, it's called e2e_stark_proof) and should actually be deleted. Since the aggregation now outputs the root proof, the correct verifier is generated here: https://github.com/scroll-tech/openvm/blob/main/crates/sdk/src/lib.rs#L280-L296
hero78119
left a comment
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.
Try new api in this PR on https://github.com/scroll-tech/ceno-reth-benchmark e2e, need to adjust further of new api design
c5f9590 to
bb3eb54
Compare
hero78119
left a comment
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.
Reported with e2e test with block 23587691
- e2e prove/verify pass
- final root proof size: 671K
- currently only cpu mode, as I didn't find respective gpu config for root prover. On cpu root proof take
89.78s. We need a way to switch to gpu config
Uh oh!
There was an error while loading. Please reload this page.