-
Notifications
You must be signed in to change notification settings - Fork 265
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
feat: Goblin acir composer #4112
Conversation
Benchmark resultsMetrics with a significant change:
Detailed resultsAll benchmarks are run on txs on the This benchmark source data is available in JSON format on S3 here. Values are compared against data from master at commit L2 block published to L1Each column represents the number of txs on an L2 block published to L1.
L2 chain processingEach column represents the number of blocks on the L2 chain where each block has 16 txs.
Circuits statsStats on running time and I/O sizes collected for every circuit run across all benchmarks.
Tree insertion statsThe duration to insert a fixed batch of leaves into each tree type.
MiscellaneousTransaction sizes based on how many contracts are deployed in the tx.
Transaction processing duration by data writes.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code is related to the c binds and is autogenerated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code in this file is autogenerated based on c binds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a very straightforward addition following existing patterns ✓
@@ -26,6 +27,11 @@ WASM_EXPORT void acir_new_acir_composer(uint32_t const* size_hint, out_ptr out) | |||
*out = new acir_proofs::AcirComposer(ntohl(*size_hint)); | |||
} | |||
|
|||
WASM_EXPORT void acir_new_goblin_acir_composer(out_ptr out) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You love to see it.
🤖 I have created a release *beep* *boop* --- <details><summary>aztec-packages: 0.20.0</summary> ## [0.20.0](aztec-packages-v0.19.0...aztec-packages-v0.20.0) (2024-01-22) ### ⚠ BREAKING CHANGES * nullifier key ([#4166](#4166)) * Unify ABIs between nargo and yarn-project ([#3989](#3989)) ### Features * **avm:** Add internal jump and return, adjust control flow ([#4140](#4140)) ([b77afb1](b77afb1)) * **avm:** Better field arithmetic ([#4142](#4142)) ([7308e31](7308e31)) * **avm:** Encode TS AVM instructions as bytecode, especially for testing ([#4115](#4115)) ([de6e2ed](de6e2ed)) * **avm:** Improve interpreter errors and tests ([#4173](#4173)) ([f0fb594](f0fb594)) * Benchmark commit function ([#4178](#4178)) ([ea84085](ea84085)) * Goblin acir composer ([#4112](#4112)) ([5e85b92](5e85b92)) * Nullifier key ([#4166](#4166)) ([7c07665](7c07665)) * **public-vm:** Avm journal ([#3945](#3945)) ([5658468](5658468)) * Publish block body separately ([#4118](#4118)) ([a04e1e3](a04e1e3)), closes [#3944](#3944) * Unify ABIs between nargo and yarn-project ([#3989](#3989)) ([d083438](d083438)) * Update noir ([#4082](#4082)) ([0e6037a](0e6037a)) * Updating L2 Block encoding and `Rollup.process` function ([#4015](#4015)) ([2d8eb37](2d8eb37)), closes [#3936](#3936) [#4010](#4010) [#4011](#4011) ### Bug Fixes * Bb.js version in yarn lockfile ([7b96760](7b96760)) * **build:** Publish bb.js from CCI ([#4151](#4151)) ([09dbfcd](09dbfcd)) * Make CMake version warning fatal ([#4144](#4144)) ([b1443fa](b1443fa)) * Misleading error message in `PublicState::read` ([#4149](#4149)) ([fa4d919](fa4d919)) * Nargo destination path in bootstrap cache ([#4103](#4103)) ([4901309](4901309)) * Reinstate Ultra arith rec verifier test ([#3886](#3886)) ([995973b](995973b)) * Upload_benchmarks_to_s3.sh missing exit ([#4046](#4046)) ([52a9327](52a9327)) ### Miscellaneous * Archiver store ([#3966](#3966)) ([af2be87](af2be87)) * **avm:** List avm opcodes in a (enum => class) map in TS ([#4113](#4113)) ([dee564a](dee564a)) * **bb:** More concise namespaces, plookup => bb::plookup ([#4146](#4146)) ([14d39ed](14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([#4117](#4117)) ([cd2f67f](cd2f67f)) * **bb:** Namespace proof_system=>bb ([#4116](#4116)) ([7438db3](7438db3)) * **docs:** Aztec-up doesnt need `latest`, remove warnings around sandbox/cli npm pkgs ([#4138](#4138)) ([2bbf7a9](2bbf7a9)) * **docs:** Update js release notes for 0.18.0 ([#4051](#4051)) ([bdbe963](bdbe963)) * **docs:** Update lsp install instructions ([#4110](#4110)) ([3138816](3138816)), closes [#4098](#4098) * Dont mirror build-system mirror_repos.yml ([#4067](#4067)) ([04f8e0d](04f8e0d)) * Fixes many broken urls ([#4109](#4109)) ([41ae75c](41ae75c)) * Remove dependency cycles in `sequencer-client` ([#4017](#4017)) ([fe4538b](fe4538b)) * Remove lodash times in favor of foundation fn ([#3877](#3877)) ([a10eef0](a10eef0)) * Remove mutex dependency ([#4160](#4160)) ([3b82be0](3b82be0)) * Remove unnecessary computation ([#4133](#4133)) ([f35bdb8](f35bdb8)) * Remove unused noir-version json ([#4105](#4105)) ([afca819](afca819)) * Remove unwanted submodules ([#4085](#4085)) ([dda7c9c](dda7c9c)) * Replace relative paths to noir-protocol-circuits ([59feeb5](59feeb5)) * Replace relative paths to noir-protocol-circuits ([44d9136](44d9136)) * Replace relative paths to noir-protocol-circuits ([84b0bad](84b0bad)) * Simplify and fix DocsExample contract, e2e singleton + codegen to not show internal methods ([#4169](#4169)) ([38d262e](38d262e)) * Update noir ([#4168](#4168)) ([d40ad06](d40ad06)) ### Documentation * Update migration notes ([#4175](#4175)) ([dbc8174](dbc8174)) * **yellow-paper:** Update circuit sections for nullifier keys and static calls ([#4155](#4155)) ([ed71a57](ed71a57)) * **yellowpaper:** Refresh of avm instruction set ([#4081](#4081)) ([52162ee](52162ee)) </details> <details><summary>barretenberg.js: 0.20.0</summary> ## [0.20.0](barretenberg.js-v0.19.0...barretenberg.js-v0.20.0) (2024-01-22) ### Features * Goblin acir composer ([#4112](#4112)) ([5e85b92](5e85b92)) </details> <details><summary>barretenberg: 0.20.0</summary> ## [0.20.0](barretenberg-v0.19.0...barretenberg-v0.20.0) (2024-01-22) ### Features * Benchmark commit function ([#4178](#4178)) ([ea84085](ea84085)) * Goblin acir composer ([#4112](#4112)) ([5e85b92](5e85b92)) ### Bug Fixes * Make CMake version warning fatal ([#4144](#4144)) ([b1443fa](b1443fa)) * Reinstate Ultra arith rec verifier test ([#3886](#3886)) ([995973b](995973b)) * Upload_benchmarks_to_s3.sh missing exit ([#4046](#4046)) ([52a9327](52a9327)) ### Miscellaneous * **bb:** More concise namespaces, plookup => bb::plookup ([#4146](#4146)) ([14d39ed](14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([#4117](#4117)) ([cd2f67f](cd2f67f)) * **bb:** Namespace proof_system=>bb ([#4116](#4116)) ([7438db3](7438db3)) * Remove mutex dependency ([#4160](#4160)) ([3b82be0](3b82be0)) * Remove unwanted submodules ([#4085](#4085)) ([dda7c9c](dda7c9c)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- <details><summary>aztec-packages: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@aztec-packages-v0.19.0...aztec-packages-v0.20.0) (2024-01-22) ### ⚠ BREAKING CHANGES * nullifier key ([#4166](AztecProtocol/aztec-packages#4166)) * Unify ABIs between nargo and yarn-project ([#3989](AztecProtocol/aztec-packages#3989)) ### Features * **avm:** Add internal jump and return, adjust control flow ([#4140](AztecProtocol/aztec-packages#4140)) ([b77afb1](AztecProtocol/aztec-packages@b77afb1)) * **avm:** Better field arithmetic ([#4142](AztecProtocol/aztec-packages#4142)) ([7308e31](AztecProtocol/aztec-packages@7308e31)) * **avm:** Encode TS AVM instructions as bytecode, especially for testing ([#4115](AztecProtocol/aztec-packages#4115)) ([de6e2ed](AztecProtocol/aztec-packages@de6e2ed)) * **avm:** Improve interpreter errors and tests ([#4173](AztecProtocol/aztec-packages#4173)) ([f0fb594](AztecProtocol/aztec-packages@f0fb594)) * Benchmark commit function ([#4178](AztecProtocol/aztec-packages#4178)) ([ea84085](AztecProtocol/aztec-packages@ea84085)) * Goblin acir composer ([#4112](AztecProtocol/aztec-packages#4112)) ([5e85b92](AztecProtocol/aztec-packages@5e85b92)) * Nullifier key ([#4166](AztecProtocol/aztec-packages#4166)) ([7c07665](AztecProtocol/aztec-packages@7c07665)) * **public-vm:** Avm journal ([#3945](AztecProtocol/aztec-packages#3945)) ([5658468](AztecProtocol/aztec-packages@5658468)) * Publish block body separately ([#4118](AztecProtocol/aztec-packages#4118)) ([a04e1e3](AztecProtocol/aztec-packages@a04e1e3)), closes [#3944](AztecProtocol/aztec-packages#3944) * Unify ABIs between nargo and yarn-project ([#3989](AztecProtocol/aztec-packages#3989)) ([d083438](AztecProtocol/aztec-packages@d083438)) * Update noir ([#4082](AztecProtocol/aztec-packages#4082)) ([0e6037a](AztecProtocol/aztec-packages@0e6037a)) * Updating L2 Block encoding and `Rollup.process` function ([#4015](AztecProtocol/aztec-packages#4015)) ([2d8eb37](AztecProtocol/aztec-packages@2d8eb37)), closes [#3936](AztecProtocol/aztec-packages#3936) [#4010](AztecProtocol/aztec-packages#4010) [#4011](AztecProtocol/aztec-packages#4011) ### Bug Fixes * Bb.js version in yarn lockfile ([7b96760](AztecProtocol/aztec-packages@7b96760)) * **build:** Publish bb.js from CCI ([#4151](AztecProtocol/aztec-packages#4151)) ([09dbfcd](AztecProtocol/aztec-packages@09dbfcd)) * Make CMake version warning fatal ([#4144](AztecProtocol/aztec-packages#4144)) ([b1443fa](AztecProtocol/aztec-packages@b1443fa)) * Misleading error message in `PublicState::read` ([#4149](AztecProtocol/aztec-packages#4149)) ([fa4d919](AztecProtocol/aztec-packages@fa4d919)) * Nargo destination path in bootstrap cache ([#4103](AztecProtocol/aztec-packages#4103)) ([4901309](AztecProtocol/aztec-packages@4901309)) * Reinstate Ultra arith rec verifier test ([#3886](AztecProtocol/aztec-packages#3886)) ([995973b](AztecProtocol/aztec-packages@995973b)) * Upload_benchmarks_to_s3.sh missing exit ([#4046](AztecProtocol/aztec-packages#4046)) ([52a9327](AztecProtocol/aztec-packages@52a9327)) ### Miscellaneous * Archiver store ([#3966](AztecProtocol/aztec-packages#3966)) ([af2be87](AztecProtocol/aztec-packages@af2be87)) * **avm:** List avm opcodes in a (enum => class) map in TS ([#4113](AztecProtocol/aztec-packages#4113)) ([dee564a](AztecProtocol/aztec-packages@dee564a)) * **bb:** More concise namespaces, plookup => bb::plookup ([#4146](AztecProtocol/aztec-packages#4146)) ([14d39ed](AztecProtocol/aztec-packages@14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([#4117](AztecProtocol/aztec-packages#4117)) ([cd2f67f](AztecProtocol/aztec-packages@cd2f67f)) * **bb:** Namespace proof_system=>bb ([#4116](AztecProtocol/aztec-packages#4116)) ([7438db3](AztecProtocol/aztec-packages@7438db3)) * **docs:** Aztec-up doesnt need `latest`, remove warnings around sandbox/cli npm pkgs ([#4138](AztecProtocol/aztec-packages#4138)) ([2bbf7a9](AztecProtocol/aztec-packages@2bbf7a9)) * **docs:** Update js release notes for 0.18.0 ([#4051](AztecProtocol/aztec-packages#4051)) ([bdbe963](AztecProtocol/aztec-packages@bdbe963)) * **docs:** Update lsp install instructions ([#4110](AztecProtocol/aztec-packages#4110)) ([3138816](AztecProtocol/aztec-packages@3138816)), closes [#4098](AztecProtocol/aztec-packages#4098) * Dont mirror build-system mirror_repos.yml ([#4067](AztecProtocol/aztec-packages#4067)) ([04f8e0d](AztecProtocol/aztec-packages@04f8e0d)) * Fixes many broken urls ([#4109](AztecProtocol/aztec-packages#4109)) ([41ae75c](AztecProtocol/aztec-packages@41ae75c)) * Remove dependency cycles in `sequencer-client` ([#4017](AztecProtocol/aztec-packages#4017)) ([fe4538b](AztecProtocol/aztec-packages@fe4538b)) * Remove lodash times in favor of foundation fn ([#3877](AztecProtocol/aztec-packages#3877)) ([a10eef0](AztecProtocol/aztec-packages@a10eef0)) * Remove mutex dependency ([#4160](AztecProtocol/aztec-packages#4160)) ([3b82be0](AztecProtocol/aztec-packages@3b82be0)) * Remove unnecessary computation ([#4133](AztecProtocol/aztec-packages#4133)) ([f35bdb8](AztecProtocol/aztec-packages@f35bdb8)) * Remove unused noir-version json ([#4105](AztecProtocol/aztec-packages#4105)) ([afca819](AztecProtocol/aztec-packages@afca819)) * Remove unwanted submodules ([#4085](AztecProtocol/aztec-packages#4085)) ([dda7c9c](AztecProtocol/aztec-packages@dda7c9c)) * Replace relative paths to noir-protocol-circuits ([59feeb5](AztecProtocol/aztec-packages@59feeb5)) * Replace relative paths to noir-protocol-circuits ([44d9136](AztecProtocol/aztec-packages@44d9136)) * Replace relative paths to noir-protocol-circuits ([84b0bad](AztecProtocol/aztec-packages@84b0bad)) * Simplify and fix DocsExample contract, e2e singleton + codegen to not show internal methods ([#4169](AztecProtocol/aztec-packages#4169)) ([38d262e](AztecProtocol/aztec-packages@38d262e)) * Update noir ([#4168](AztecProtocol/aztec-packages#4168)) ([d40ad06](AztecProtocol/aztec-packages@d40ad06)) ### Documentation * Update migration notes ([#4175](AztecProtocol/aztec-packages#4175)) ([dbc8174](AztecProtocol/aztec-packages@dbc8174)) * **yellow-paper:** Update circuit sections for nullifier keys and static calls ([#4155](AztecProtocol/aztec-packages#4155)) ([ed71a57](AztecProtocol/aztec-packages@ed71a57)) * **yellowpaper:** Refresh of avm instruction set ([#4081](AztecProtocol/aztec-packages#4081)) ([52162ee](AztecProtocol/aztec-packages@52162ee)) </details> <details><summary>barretenberg.js: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@barretenberg.js-v0.19.0...barretenberg.js-v0.20.0) (2024-01-22) ### Features * Goblin acir composer ([#4112](AztecProtocol/aztec-packages#4112)) ([5e85b92](AztecProtocol/aztec-packages@5e85b92)) </details> <details><summary>barretenberg: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@barretenberg-v0.19.0...barretenberg-v0.20.0) (2024-01-22) ### Features * Benchmark commit function ([#4178](AztecProtocol/aztec-packages#4178)) ([ea84085](AztecProtocol/aztec-packages@ea84085)) * Goblin acir composer ([#4112](AztecProtocol/aztec-packages#4112)) ([5e85b92](AztecProtocol/aztec-packages@5e85b92)) ### Bug Fixes * Make CMake version warning fatal ([#4144](AztecProtocol/aztec-packages#4144)) ([b1443fa](AztecProtocol/aztec-packages@b1443fa)) * Reinstate Ultra arith rec verifier test ([#3886](AztecProtocol/aztec-packages#3886)) ([995973b](AztecProtocol/aztec-packages@995973b)) * Upload_benchmarks_to_s3.sh missing exit ([#4046](AztecProtocol/aztec-packages#4046)) ([52a9327](AztecProtocol/aztec-packages@52a9327)) ### Miscellaneous * **bb:** More concise namespaces, plookup => bb::plookup ([#4146](AztecProtocol/aztec-packages#4146)) ([14d39ed](AztecProtocol/aztec-packages@14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([#4117](AztecProtocol/aztec-packages#4117)) ([cd2f67f](AztecProtocol/aztec-packages@cd2f67f)) * **bb:** Namespace proof_system=>bb ([#4116](AztecProtocol/aztec-packages#4116)) ([7438db3](AztecProtocol/aztec-packages@7438db3)) * Remove mutex dependency ([#4160](AztecProtocol/aztec-packages#4160)) ([3b82be0](AztecProtocol/aztec-packages@3b82be0)) * Remove unwanted submodules ([#4085](AztecProtocol/aztec-packages#4085)) ([dda7c9c](AztecProtocol/aztec-packages@dda7c9c)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
A bit of background: At the end of 2023, we had a small subset of the acir_tests passing for GUH+Goblin. Early work in 2024 got all of the tests passing for GUH. (At this stage we were basically running all acir_tests for UltraPlonk and GoblinUltraHonk). I turned the Goblin component off because tests were failing in an unpredictable way. (I now think this is a deterministic issue with the Translator that was showing up more prominently due to running the Translator hundreds of times via the acir_tests suite). This work maintains testing of all the noir programs in the acir_test suite for GUH, but adds back in a full Goblin workflow for a single arbitrary test. This is done through two workflows: a "goblin accumulate" workflow (for constructing/verifying GUH proofs) and a "full goblin" flow (for additionally constructing and verifying ECCVM/Translator proofs). (Note: this latter flow will eventually also involve merge logic but that's still not incorporated in this work). To make things cleaner, I've introduced a `GoblinAcirComposer` to handle all Goblin/Honk logic, analogous to the `AcirComposer` for UltraPlonk. Note: there is still no mechanism for creating goblin ops directly from acir opcodes. (Eventually this will just happen via a blackbox call to some recursive verification / folding operation for Honk/Goblin). The OpQueue being processed by the ECCVM/Translator is just a random set of ops "appended" to the circuit in question.
🤖 I have created a release *beep* *boop* --- <details><summary>aztec-packages: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@aztec-packages-v0.19.0...aztec-packages-v0.20.0) (2024-01-22) ### ⚠ BREAKING CHANGES * nullifier key ([AztecProtocol#4166](AztecProtocol#4166)) * Unify ABIs between nargo and yarn-project ([AztecProtocol#3989](AztecProtocol#3989)) ### Features * **avm:** Add internal jump and return, adjust control flow ([AztecProtocol#4140](AztecProtocol#4140)) ([b77afb1](AztecProtocol@b77afb1)) * **avm:** Better field arithmetic ([AztecProtocol#4142](AztecProtocol#4142)) ([7308e31](AztecProtocol@7308e31)) * **avm:** Encode TS AVM instructions as bytecode, especially for testing ([AztecProtocol#4115](AztecProtocol#4115)) ([de6e2ed](AztecProtocol@de6e2ed)) * **avm:** Improve interpreter errors and tests ([AztecProtocol#4173](AztecProtocol#4173)) ([f0fb594](AztecProtocol@f0fb594)) * Benchmark commit function ([AztecProtocol#4178](AztecProtocol#4178)) ([ea84085](AztecProtocol@ea84085)) * Goblin acir composer ([AztecProtocol#4112](AztecProtocol#4112)) ([5e85b92](AztecProtocol@5e85b92)) * Nullifier key ([AztecProtocol#4166](AztecProtocol#4166)) ([7c07665](AztecProtocol@7c07665)) * **public-vm:** Avm journal ([AztecProtocol#3945](AztecProtocol#3945)) ([5658468](AztecProtocol@5658468)) * Publish block body separately ([AztecProtocol#4118](AztecProtocol#4118)) ([a04e1e3](AztecProtocol@a04e1e3)), closes [AztecProtocol#3944](AztecProtocol#3944) * Unify ABIs between nargo and yarn-project ([AztecProtocol#3989](AztecProtocol#3989)) ([d083438](AztecProtocol@d083438)) * Update noir ([AztecProtocol#4082](AztecProtocol#4082)) ([0e6037a](AztecProtocol@0e6037a)) * Updating L2 Block encoding and `Rollup.process` function ([AztecProtocol#4015](AztecProtocol#4015)) ([2d8eb37](AztecProtocol@2d8eb37)), closes [AztecProtocol#3936](AztecProtocol#3936) [AztecProtocol#4010](AztecProtocol#4010) [AztecProtocol#4011](AztecProtocol#4011) ### Bug Fixes * Bb.js version in yarn lockfile ([7b96760](AztecProtocol@7b96760)) * **build:** Publish bb.js from CCI ([AztecProtocol#4151](AztecProtocol#4151)) ([09dbfcd](AztecProtocol@09dbfcd)) * Make CMake version warning fatal ([AztecProtocol#4144](AztecProtocol#4144)) ([b1443fa](AztecProtocol@b1443fa)) * Misleading error message in `PublicState::read` ([AztecProtocol#4149](AztecProtocol#4149)) ([fa4d919](AztecProtocol@fa4d919)) * Nargo destination path in bootstrap cache ([AztecProtocol#4103](AztecProtocol#4103)) ([4901309](AztecProtocol@4901309)) * Reinstate Ultra arith rec verifier test ([AztecProtocol#3886](AztecProtocol#3886)) ([995973b](AztecProtocol@995973b)) * Upload_benchmarks_to_s3.sh missing exit ([AztecProtocol#4046](AztecProtocol#4046)) ([52a9327](AztecProtocol@52a9327)) ### Miscellaneous * Archiver store ([AztecProtocol#3966](AztecProtocol#3966)) ([af2be87](AztecProtocol@af2be87)) * **avm:** List avm opcodes in a (enum => class) map in TS ([AztecProtocol#4113](AztecProtocol#4113)) ([dee564a](AztecProtocol@dee564a)) * **bb:** More concise namespaces, plookup => bb::plookup ([AztecProtocol#4146](AztecProtocol#4146)) ([14d39ed](AztecProtocol@14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([AztecProtocol#4117](AztecProtocol#4117)) ([cd2f67f](AztecProtocol@cd2f67f)) * **bb:** Namespace proof_system=>bb ([AztecProtocol#4116](AztecProtocol#4116)) ([7438db3](AztecProtocol@7438db3)) * **docs:** Aztec-up doesnt need `latest`, remove warnings around sandbox/cli npm pkgs ([AztecProtocol#4138](AztecProtocol#4138)) ([2bbf7a9](AztecProtocol@2bbf7a9)) * **docs:** Update js release notes for 0.18.0 ([AztecProtocol#4051](AztecProtocol#4051)) ([bdbe963](AztecProtocol@bdbe963)) * **docs:** Update lsp install instructions ([AztecProtocol#4110](AztecProtocol#4110)) ([3138816](AztecProtocol@3138816)), closes [AztecProtocol#4098](AztecProtocol#4098) * Dont mirror build-system mirror_repos.yml ([AztecProtocol#4067](AztecProtocol#4067)) ([04f8e0d](AztecProtocol@04f8e0d)) * Fixes many broken urls ([AztecProtocol#4109](AztecProtocol#4109)) ([41ae75c](AztecProtocol@41ae75c)) * Remove dependency cycles in `sequencer-client` ([AztecProtocol#4017](AztecProtocol#4017)) ([fe4538b](AztecProtocol@fe4538b)) * Remove lodash times in favor of foundation fn ([AztecProtocol#3877](AztecProtocol#3877)) ([a10eef0](AztecProtocol@a10eef0)) * Remove mutex dependency ([AztecProtocol#4160](AztecProtocol#4160)) ([3b82be0](AztecProtocol@3b82be0)) * Remove unnecessary computation ([AztecProtocol#4133](AztecProtocol#4133)) ([f35bdb8](AztecProtocol@f35bdb8)) * Remove unused noir-version json ([AztecProtocol#4105](AztecProtocol#4105)) ([afca819](AztecProtocol@afca819)) * Remove unwanted submodules ([AztecProtocol#4085](AztecProtocol#4085)) ([dda7c9c](AztecProtocol@dda7c9c)) * Replace relative paths to noir-protocol-circuits ([59feeb5](AztecProtocol@59feeb5)) * Replace relative paths to noir-protocol-circuits ([44d9136](AztecProtocol@44d9136)) * Replace relative paths to noir-protocol-circuits ([84b0bad](AztecProtocol@84b0bad)) * Simplify and fix DocsExample contract, e2e singleton + codegen to not show internal methods ([AztecProtocol#4169](AztecProtocol#4169)) ([38d262e](AztecProtocol@38d262e)) * Update noir ([AztecProtocol#4168](AztecProtocol#4168)) ([d40ad06](AztecProtocol@d40ad06)) ### Documentation * Update migration notes ([AztecProtocol#4175](AztecProtocol#4175)) ([dbc8174](AztecProtocol@dbc8174)) * **yellow-paper:** Update circuit sections for nullifier keys and static calls ([AztecProtocol#4155](AztecProtocol#4155)) ([ed71a57](AztecProtocol@ed71a57)) * **yellowpaper:** Refresh of avm instruction set ([AztecProtocol#4081](AztecProtocol#4081)) ([52162ee](AztecProtocol@52162ee)) </details> <details><summary>barretenberg.js: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@barretenberg.js-v0.19.0...barretenberg.js-v0.20.0) (2024-01-22) ### Features * Goblin acir composer ([AztecProtocol#4112](AztecProtocol#4112)) ([5e85b92](AztecProtocol@5e85b92)) </details> <details><summary>barretenberg: 0.20.0</summary> ## [0.20.0](AztecProtocol/aztec-packages@barretenberg-v0.19.0...barretenberg-v0.20.0) (2024-01-22) ### Features * Benchmark commit function ([AztecProtocol#4178](AztecProtocol#4178)) ([ea84085](AztecProtocol@ea84085)) * Goblin acir composer ([AztecProtocol#4112](AztecProtocol#4112)) ([5e85b92](AztecProtocol@5e85b92)) ### Bug Fixes * Make CMake version warning fatal ([AztecProtocol#4144](AztecProtocol#4144)) ([b1443fa](AztecProtocol@b1443fa)) * Reinstate Ultra arith rec verifier test ([AztecProtocol#3886](AztecProtocol#3886)) ([995973b](AztecProtocol@995973b)) * Upload_benchmarks_to_s3.sh missing exit ([AztecProtocol#4046](AztecProtocol#4046)) ([52a9327](AztecProtocol@52a9327)) ### Miscellaneous * **bb:** More concise namespaces, plookup => bb::plookup ([AztecProtocol#4146](AztecProtocol#4146)) ([14d39ed](AztecProtocol@14d39ed)) * **bb:** Namespace plonk::stdlib => stdlib ([AztecProtocol#4117](AztecProtocol#4117)) ([cd2f67f](AztecProtocol@cd2f67f)) * **bb:** Namespace proof_system=>bb ([AztecProtocol#4116](AztecProtocol#4116)) ([7438db3](AztecProtocol@7438db3)) * Remove mutex dependency ([AztecProtocol#4160](AztecProtocol#4160)) ([3b82be0](AztecProtocol@3b82be0)) * Remove unwanted submodules ([AztecProtocol#4085](AztecProtocol#4085)) ([dda7c9c](AztecProtocol@dda7c9c)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
A bit of background: At the end of 2023, we had a small subset of the acir_tests passing for GUH+Goblin. Early work in 2024 got all of the tests passing for GUH. (At this stage we were basically running all acir_tests for UltraPlonk and GoblinUltraHonk). I turned the Goblin component off because tests were failing in an unpredictable way. (I now think this is a deterministic issue with the Translator that was showing up more prominently due to running the Translator hundreds of times via the acir_tests suite). This work maintains testing of all the noir programs in the acir_test suite for GUH, but adds back in a full Goblin workflow for a single arbitrary test. This is done through two workflows: a "goblin accumulate" workflow (for constructing/verifying GUH proofs) and a "full goblin" flow (for additionally constructing and verifying ECCVM/Translator proofs). (Note: this latter flow will eventually also involve merge logic but that's still not incorporated in this work).
To make things cleaner, I've introduced a
GoblinAcirComposer
to handle all Goblin/Honk logic, analogous to theAcirComposer
for UltraPlonk.Note: there is still no mechanism for creating goblin ops directly from acir opcodes. (Eventually this will just happen via a blackbox call to some recursive verification / folding operation for Honk/Goblin). The OpQueue being processed by the ECCVM/Translator is just a random set of ops "appended" to the circuit in question.