Skip to content

chore: remove chonk bench once and for all#23067

Merged
ludamad merged 1 commit into
merge-train/barretenbergfrom
lde/no-more-chonk-bench
May 8, 2026
Merged

chore: remove chonk bench once and for all#23067
ludamad merged 1 commit into
merge-train/barretenbergfrom
lde/no-more-chonk-bench

Conversation

@ledwards2225

Copy link
Copy Markdown
Contributor

Remove the chonk_bench as the risk of using it to make decisions is greater than the utility it provides.

/**
* @brief Benchmark BN254 G1 point decompression (used by SRS compressed download)
*/
void bn254_point_decompression(benchmark::State& state)

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.

keeping this (formerly located in chonk bench for some reason)

@ledwards2225 ledwards2225 marked this pull request as ready for review May 7, 2026 20:56
@ledwards2225 ledwards2225 enabled auto-merge (squash) May 7, 2026 21:36
@ledwards2225 ledwards2225 disabled auto-merge May 7, 2026 21:36
@ledwards2225 ledwards2225 enabled auto-merge (squash) May 7, 2026 21:36
@ludamad ludamad disabled auto-merge May 8, 2026 01:24
@ludamad ludamad merged commit 7d71424 into merge-train/barretenberg May 8, 2026
22 checks passed
@ludamad ludamad deleted the lde/no-more-chonk-bench branch May 8, 2026 01:24
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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants