Skip to content

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Sep 2, 2025

BEGIN_COMMIT_OVERRIDE
feat(bb): --print_bench (#16673)
refactor(bb): remove --output_format, have one-size-fits-all format (#16201)
fix(bb): fix vk simplified buffer (#16720)
fix(avm): avm vk after serialization unification (#16721)
END_COMMIT_OVERRIDE

While the pieces were there before, I would say the culmination of them amounts to a new feature. Now if you do --print_bench or BB_BENCH=1 on a bb or a test (or even a bench to get clearer numbers) you will get a listing like below
@ludamad ludamad added this pull request to the merge queue Sep 2, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Sep 2, 2025
ludamad and others added 3 commits September 2, 2025 16:09
…16201)

This changes vks to be serialized as bytes using to_field_elements. Proofs were already byte-compatible, this allows having bb not worry about how to split fields into JSON

The changes are holistic as we now need to split the fields from the binary vk/proof - but this is now trivial with the new deserialization format.
@ludamad ludamad requested a review from LeilaWang as a code owner September 2, 2025 21:00
@ludamad ludamad added the ci-full Run all master checks. label Sep 2, 2025
@ludamad ludamad enabled auto-merge September 3, 2025 01:07
@ludamad ludamad added this pull request to the merge queue Sep 3, 2025
Merged via the queue into next with commit 6107786 Sep 3, 2025
17 checks passed
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-no-squash

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants