Skip to content

fix: increase WASM CRS size to 2^21 for chonk proofs#21194

Draft
AztecBot wants to merge 1 commit intomerge-train/barretenbergfrom
claudebox/fix-chonk-wasm-crs-size
Draft

fix: increase WASM CRS size to 2^21 for chonk proofs#21194
AztecBot wants to merge 1 commit intomerge-train/barretenbergfrom
claudebox/fix-chonk-wasm-crs-size

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 5, 2026

The chonk prover now needs 2^21 CRS points but initSRSChonk defaulted to getDefaultSrsSize() which returns 2^20. This caused all Wasm-backend chonk integration tests to fail with:

prover trying to get too many points in MemBn254CrsFactory! 1048576 vs 2097152

The native backend was unaffected since it loads CRS from disk on demand.

Fix: hardcode initSRSChonk default to 2 ** 21 instead of using getDefaultSrsSize() (which is sized for general use and has iOS-specific constraints).

ClaudeBox log: http://ci.aztec-labs.com/db5d7cce1bf23d04-1

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 5, 2026
@ludamad ludamad self-requested a review March 5, 2026 23:49
Copy link
Collaborator

@ludamad ludamad left a comment

Choose a reason for hiding this comment

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

we shouldnt be bumping here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants