Skip to content

chore: bump bb to 0.87.0 #8506

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
May 21, 2025
Merged

chore: bump bb to 0.87.0 #8506

merged 12 commits into from
May 21, 2025

Conversation

TomAFrench
Copy link
Member

@TomAFrench TomAFrench commented May 14, 2025

Description

Problem*

Resolves

Summary*

This PR bumps us up to a newer bb version to benefit from the crypto team's optimizations.

I've spoken to Lucas about the regressions (on empty circuits!) and it's expected and they're working on it.

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@TomAFrench TomAFrench requested a review from a team May 14, 2025 13:48
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Compilation Time'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 5ffe28e Previous: d919e6a Ratio
semaphore-depth-10 1.023 s 0.843 s 1.21

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench

github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request May 15, 2025
This PR fixes the issue in bb.js dependencies which is causing failures
in noir-lang/noir#8506

This was broken in
#13590 where the
existing msgpack dependency was removed.

Co-authored-by: AztecBot <[email protected]>
AztecBot pushed a commit to AztecProtocol/barretenberg that referenced this pull request May 16, 2025
This PR fixes the issue in bb.js dependencies which is causing failures
in noir-lang/noir#8506

This was broken in
AztecProtocol/aztec-packages#13590 where the
existing msgpack dependency was removed.

Co-authored-by: AztecBot <[email protected]>
@TomAFrench TomAFrench changed the title chore: bump bb to 0.86.0 chore: bump bb to 0.87.0 May 20, 2025
@TomAFrench TomAFrench enabled auto-merge May 21, 2025 13:14
@TomAFrench TomAFrench added this pull request to the merge queue May 21, 2025
Merged via the queue into master with commit c89efcb May 21, 2025
118 checks passed
@TomAFrench TomAFrench deleted the bump-bb branch May 21, 2025 14:17
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