Skip to content

feat!: optimized Poseidon2 #22652

Merged
iakovenkos merged 48 commits into
merge-train/barretenbergfrom
si/poseidon2-opt-attempt
May 8, 2026
Merged

feat!: optimized Poseidon2 #22652
iakovenkos merged 48 commits into
merge-train/barretenbergfrom
si/poseidon2-opt-attempt

Conversation

@iakovenkos

Copy link
Copy Markdown
Contributor

.

@iakovenkos iakovenkos requested a review from LeilaWang as a code owner April 18, 2026 07:22
@iakovenkos iakovenkos self-assigned this Apr 18, 2026
@iakovenkos iakovenkos added the ci-full Run all master checks. label Apr 18, 2026
AztecBot added a commit that referenced this pull request Apr 20, 2026
- CHONK_PROOF_LENGTH: 1332 -> 1317 (MegaZKFlavor::NUM_ALL_ENTITIES delta
  from databus genericization on merge-train after PR #22652's base)
  - mock_verifier_inputs.test.cpp static_assert
  - noir constants.nr
  - yarn-project constants.gen.ts
- hypernova_verifier.test.cpp: drop stale {CALLDATA,SECONDARY_CALLDATA,
  RETURN_DATA}_READ_TAGS entries from Round 0 of expected manifest
  (databus no longer sends read_tags per-bus-column)
- rollup-tx-base-private/Prover.toml: trim 15 trailing zero entries from
  inputs.hiding_kernel_proof_data.proof (1332 -> 1317)

bb tests: 61/61 binaries pass (552/552 dsl_tests with 7 pre-existing
GTEST_SKIP; pinned VK-hash script not run per request)
@iakovenkos iakovenkos removed the ci-full Run all master checks. label May 4, 2026
- The **Poseidon2 permutation**, i.e. the round function used by the sponge.
- **Circuit custom gate relations** that enforce the permutation’s correctness.

## Contents

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@notnotraju here's the updated readme

@iakovenkos iakovenkos changed the title chore: P2 explorations feat!: optimized Poseidon2 May 5, 2026
@iakovenkos iakovenkos requested review from notnotraju May 5, 2026 20:13

@notnotraju notnotraju left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thanks! I made a few additions to the documentation to make it easier for me to understand and added a few tests.

@iakovenkos iakovenkos added the ci-full Run all master checks. label May 8, 2026
@iakovenkos iakovenkos enabled auto-merge (squash) May 8, 2026 12:31
@iakovenkos iakovenkos merged commit 3827e14 into merge-train/barretenberg May 8, 2026
14 checks passed
@iakovenkos iakovenkos deleted the si/poseidon2-opt-attempt branch May 8, 2026 17:26
Thunkar pushed a commit that referenced this pull request May 8, 2026
## Summary

Fix CI failure on `merge-train/fairies` (run
[25572561023](https://github.com/AztecProtocol/aztec-packages/actions/runs/25572561023),
log [1778264338812903](http://ci.aztec-labs.com/1778264338812903)).

The merge of `next` into `merge-train/fairies` (78632af) brought in
barretenberg changes — `feat!: optimized Poseidon2 (#22652)` and `feat:
n1 apps (#22974)` via `feat: merge-train/barretenberg (#23025)` — which
grew bb.js by enough to push the playground main entrypoint 0.02 KB over
the 1750 KB hard limit:

```
❌ assets/index-BEz4wQVA.js: 1750.02 KB exceeds limit of 1750 KB (Main entrypoint, hard limit)
```

Bumping the limit to 1800 KB (matching the prior +50 KB increment from
the bump log) restores headroom.

## Test plan

- [ ] CI green on this PR (chunk size check passes)


ClaudeBox log: https://claudebox.work/s/22af49d0486fc035?run=1
rangozd pushed a commit to rangozd/aztec-packages that referenced this pull request May 16, 2026
BEGIN_COMMIT_OVERRIDE
fix(bb): clamp BatchMergeProver degree-check loop to fix nightly debug
SIGABRT (AztecProtocol#23019)
feat: extend databus with 2 more cols (AztecProtocol#23010)
feat: n1 apps (AztecProtocol#22974)
chore: remove chonk bench once and for all (AztecProtocol#23067)
fix(bb): clamp BatchMergeProver degree-check loop to fix nightly debug
SIGABRT (AztecProtocol#23077)
feat!: optimized Poseidon2  (AztecProtocol#22652)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-full Run all master checks. ci-full-no-test-cache

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants