Skip to content

refactor(stm): applying final comments of PR #3348#3352

Open
damrobi wants to merge 3 commits into
mainfrom
damrobi/msnark/3141-fix-final-comments
Open

refactor(stm): applying final comments of PR #3348#3352
damrobi wants to merge 3 commits into
mainfrom
damrobi/msnark/3141-fix-final-comments

Conversation

@damrobi

@damrobi damrobi commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Content

This PR includes some small fixes from the leftover comments of PR #3348, mainly:

  • Better context for the IvcCircuitError::CertificateProofRejected error
  • Better handling of the GenesisMessagePreimage in tests
  • Name change for the AncillaryProverData variant

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • CHANGELOG file is updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • All check jobs of the CI have succeeded
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Comments

Issue(s)

Relates to #3141

@damrobi damrobi self-assigned this Jun 25, 2026
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Test Results

     5 files  ± 0     206 suites  ±0   1h 19m 18s ⏱️ - 1h 15m 37s
 3 261 tests  - 18   3 261 ✅  - 18  0 💤 ±0  0 ❌ ±0 
10 924 runs   - 18  10 924 ✅  - 18  0 💤 ±0  0 ❌ ±0 

Results for commit f2852c7. ± Comparison against base commit 5d25797.

This pull request removes 18 tests.
mithril-stm ‑ proof_system::halo2_snark::clerk::tests::slow::selection_distributes_fairly_across_index_range
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::different_parameters_prove_and_verify_fails
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::fails_with_empty_signatures
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::fails_with_insufficient_signatures
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::golden::golden_conversion
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::midnight_vk_wrapper_to_from_bytes
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::non_deterministic_proofs_verify
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::prepare_and_check_fails_with_random_bytes
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::prepare_and_check_fails_with_wrong_message
mithril-stm ‑ proof_system::halo2_snark::proof::tests::slow::produces_valid_snark_proof
…

♻️ This comment has been updated with latest results.

@damrobi damrobi temporarily deployed to testing-preview June 25, 2026 11:57 — with GitHub Actions Inactive
Comment thread mithril-stm/src/circuits/halo2_ivc/errors.rs Outdated
damrobi and others added 2 commits June 25, 2026 17:46
Expose the trusted setup and key cache modules on wasm and gate the
blocking SRS download behind non-wasm so the verifier-only client builds.
@damrobi damrobi temporarily deployed to testing-preview June 25, 2026 16:21 — with GitHub Actions Inactive
@damrobi damrobi marked this pull request as ready for review June 26, 2026 08:09

@jpraynaud jpraynaud left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Successfully merging this pull request may close these issues.

2 participants