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

Allow for local proving #295

Open
JossDuff opened this issue Dec 18, 2024 · 1 comment
Open

Allow for local proving #295

JossDuff opened this issue Dec 18, 2024 · 1 comment

Comments

@JossDuff
Copy link

JossDuff commented Dec 18, 2024

op-proposer has a command line arg OP_SUCCINCT_SERVER_URL that allows for requesting proofs to provers other than the proving network, but the proposer/succinct proving server middleware only allows for forwarding requests to the proving network or mock proofs. But myself and others might want to either run proofs locally for benchmarking or if we want to host our own prover.

To do some local testing and benchmarking we modified proposer/succinct/bin/server.rs and proposer/succinct/src/lib.rs to request the proof via ProverClient::prove(...) which runs the proof locally by default. The changes were pretty trivial and others might find them useful. I'd be happy to open a pr with this and any modifications that might be necessary :)

@ratankaliani
Copy link
Member

Hey! Is there a reason why you want to bench locally vs. on the proving network? OP Succinct workloads perform better on the proving network because it can scale up to meet demand horizontally. Will DM.

Feel free to put up a PR with the diff, happy to review & merge :)

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

No branches or pull requests

2 participants