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

Bonsai only supports succinct assumption receipts #1807

Open
kpp opened this issue Feb 4, 2025 · 1 comment
Open

Bonsai only supports succinct assumption receipts #1807

kpp opened this issue Feb 4, 2025 · 1 comment

Comments

@kpp
Copy link
Contributor

kpp commented Feb 4, 2025

Summary

2025-02-04T13:59:23.867364Z ERROR panic: thread '<unnamed>' panicked at 'Proof creation must not fail: Bonsai only supports succinct assumption receipts. Use ProverOpts::succinct()when proving any assumptions.': /home/kpp/chainway/secret-sovereign-sdk/crates/prover-services/src/parallel/mod.rs:166

Steps to reproduce

Run test_light_client_proving_multiple_proofs with bonsai api key set.

Expected Result

Test passes

Actual Result

Error

Additional Information

With this patch it works

index 36a31a2b..d4296e86 100644
--- a/crates/risc0/src/host.rs
+++ b/crates/risc0/src/host.rs
@@ -165,7 +165,7 @@ impl ZkvmHost for Risc0BonsaiHost {
         tracing::info!("Starting risc0 proving");
 
         let ProveInfo { receipt, stats } =
-            prover.prove_with_opts(env, &elf, &ProverOpts::groth16())?;
+            prover.prove_with_opts(env, &elf, &ProverOpts::succinct())?;
 
@eyusufatik
Copy link
Member

Looks like Bonsai doesn't accept groth16 assumptions. Let's find out why. If this is something that can't be provided we might need to look into ways to get bonsai receipt than wrap to snark ourselves.

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