-
Notifications
You must be signed in to change notification settings - Fork 233
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: Add init check by default to public fns #4897
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.
|
6d1b67a
to
521a258
Compare
This reverts commit 521a258.
541f614
to
7a41882
Compare
This reverts commit 4550f25.
🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.26.0</summary> ## [0.26.0](aztec-package-v0.25.0...aztec-package-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * move noir out of yarn-project ([#4479](#4479)) * note type ids ([#4500](#4500)) * aztec binary ([#3927](#3927)) ### Features * Aztec binary ([#3927](#3927)) ([12356d9](12356d9)) * Note type ids ([#4500](#4500)) ([e1da2fd](e1da2fd)) * Parallel native/wasm bb builds. Better messaging around using ci cache. ([#4766](#4766)) ([a924e55](a924e55)) ### Bug Fixes * Add new oracle contract to devnet in CI ([#4687](#4687)) ([920fa10](920fa10)) * Add registry contract to list ([#4694](#4694)) ([3675e1d](3675e1d)) * Aztec binary fixes ([#4273](#4273)) ([84e1f7d](84e1f7d)) * L1 contract address config ([#4684](#4684)) ([20e7605](20e7605)) * P2p-bootstrap ECS command + /status route ([#4682](#4682)) ([21ec23d](21ec23d)) * Relative LogFn import ([#4328](#4328)) ([1faead5](1faead5)) ### Miscellaneous * Lift rollup address check & deplot kv-store to npm ([#4483](#4483)) ([92d0aa4](92d0aa4)) * Move noir out of yarn-project ([#4479](#4479)) ([1fe674b](1fe674b)), closes [#4107](#4107) * Squash yp ypb + other build improvements. ([#4901](#4901)) ([be5855c](be5855c)) * Updating viem ([#4783](#4783)) ([23bc26a](23bc26a)) </details> <details><summary>barretenberg.js: 0.26.0</summary> ## [0.26.0](barretenberg.js-v0.25.0...barretenberg.js-v0.26.0) (2024-03-05) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-cli: 0.26.0</summary> ## [0.26.0](aztec-cli-v0.25.0...aztec-cli-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * Use new deployment flow in ContractDeployer ([#4497](#4497)) * move noir out of yarn-project ([#4479](#4479)) * note type ids ([#4500](#4500)) * Include contract class id in deployment info ([#4223](#4223)) * aztec binary ([#3927](#3927)) ### Features * **avm-transpiler:** Brillig to AVM transpiler ([#4227](#4227)) ([c366c6e](c366c6e)) * Aztec binary ([#3927](#3927)) ([12356d9](12356d9)) * Aztec.js API for registering a contract class ([#4469](#4469)) ([d566c74](d566c74)) * Include contract class id in deployment info ([#4223](#4223)) ([0ed4126](0ed4126)), closes [#4054](#4054) * Moving the unbox option to npx command ([#4718](#4718)) ([4c3bb92](4c3bb92)) * Note type ids ([#4500](#4500)) ([e1da2fd](e1da2fd)) * Parallel native/wasm bb builds. Better messaging around using ci cache. ([#4766](#4766)) ([a924e55](a924e55)) * Use new deployment flow in ContractDeployer ([#4497](#4497)) ([0702dc6](0702dc6)) ### Bug Fixes * Add new oracle contract to devnet in CI ([#4687](#4687)) ([920fa10](920fa10)) * Load contract artifact from json ([#4352](#4352)) ([47a0a79](47a0a79)) ### Miscellaneous * **docs:** Fix a few links to docs ([#4260](#4260)) ([1c8ea49](1c8ea49)) * Move noir out of yarn-project ([#4479](#4479)) ([1fe674b](1fe674b)), closes [#4107](#4107) * Remove stubbed docs ([#4196](#4196)) ([25a4bc4](25a4bc4)) * Squash yp ypb + other build improvements. ([#4901](#4901)) ([be5855c](be5855c)) * Updating viem ([#4783](#4783)) ([23bc26a](23bc26a)) </details> <details><summary>aztec-packages: 0.26.0</summary> ## [0.26.0](aztec-packages-v0.25.0...aztec-packages-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * Internal as a macro ([#4898](#4898)) ### Features * Add init check by default to public fns ([#4897](#4897)) ([4550f25](4550f25)) * Enable public constructor functions ([#4896](#4896)) ([7b06895](7b06895)) * Internal as a macro ([#4898](#4898)) ([73d640a](73d640a)) * We no longer update version packages via scripts ([#4962](#4962)) ([31d470b](31d470b)) ### Miscellaneous * Disable failing test temporarily ([ec61974](ec61974)) * Fixed call nesting, tests and docs ([#4932](#4932)) ([bd5c879](bd5c879)) * Specify packages individually for release-please ([#4960](#4960)) ([dddc35f](dddc35f)) * Sync noir repo ([#4947](#4947)) ([7ff9b71](7ff9b71)) * Unused vars cleanup + updated TODOs ([#4883](#4883)) ([3747619](3747619)) * Update escrow to use PrivateImmutable ([#4942](#4942)) ([245d801](245d801)) </details> <details><summary>barretenberg: 0.26.0</summary> ## [0.26.0](barretenberg-v0.25.0...barretenberg-v0.26.0) (2024-03-05) ### Miscellaneous * **barretenberg:** Synchronize aztec-packages versions </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
BEGIN_COMMIT_OVERRIDE feat: Add init check by default to public fns (AztecProtocol/aztec-packages#4897) feat: Enable public constructor functions (AztecProtocol/aztec-packages#4896) END_COMMIT_OVERRIDE --------- Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.26.0</summary> ## [0.26.0](AztecProtocol/aztec-packages@aztec-package-v0.25.0...aztec-package-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * move noir out of yarn-project ([#4479](AztecProtocol/aztec-packages#4479)) * note type ids ([#4500](AztecProtocol/aztec-packages#4500)) * aztec binary ([#3927](AztecProtocol/aztec-packages#3927)) ### Features * Aztec binary ([#3927](AztecProtocol/aztec-packages#3927)) ([12356d9](AztecProtocol/aztec-packages@12356d9)) * Note type ids ([#4500](AztecProtocol/aztec-packages#4500)) ([e1da2fd](AztecProtocol/aztec-packages@e1da2fd)) * Parallel native/wasm bb builds. Better messaging around using ci cache. ([#4766](AztecProtocol/aztec-packages#4766)) ([a924e55](AztecProtocol/aztec-packages@a924e55)) ### Bug Fixes * Add new oracle contract to devnet in CI ([#4687](AztecProtocol/aztec-packages#4687)) ([920fa10](AztecProtocol/aztec-packages@920fa10)) * Add registry contract to list ([#4694](AztecProtocol/aztec-packages#4694)) ([3675e1d](AztecProtocol/aztec-packages@3675e1d)) * Aztec binary fixes ([#4273](AztecProtocol/aztec-packages#4273)) ([84e1f7d](AztecProtocol/aztec-packages@84e1f7d)) * L1 contract address config ([#4684](AztecProtocol/aztec-packages#4684)) ([20e7605](AztecProtocol/aztec-packages@20e7605)) * P2p-bootstrap ECS command + /status route ([#4682](AztecProtocol/aztec-packages#4682)) ([21ec23d](AztecProtocol/aztec-packages@21ec23d)) * Relative LogFn import ([#4328](AztecProtocol/aztec-packages#4328)) ([1faead5](AztecProtocol/aztec-packages@1faead5)) ### Miscellaneous * Lift rollup address check & deplot kv-store to npm ([#4483](AztecProtocol/aztec-packages#4483)) ([92d0aa4](AztecProtocol/aztec-packages@92d0aa4)) * Move noir out of yarn-project ([#4479](AztecProtocol/aztec-packages#4479)) ([1fe674b](AztecProtocol/aztec-packages@1fe674b)), closes [#4107](AztecProtocol/aztec-packages#4107) * Squash yp ypb + other build improvements. ([#4901](AztecProtocol/aztec-packages#4901)) ([be5855c](AztecProtocol/aztec-packages@be5855c)) * Updating viem ([#4783](AztecProtocol/aztec-packages#4783)) ([23bc26a](AztecProtocol/aztec-packages@23bc26a)) </details> <details><summary>barretenberg.js: 0.26.0</summary> ## [0.26.0](AztecProtocol/aztec-packages@barretenberg.js-v0.25.0...barretenberg.js-v0.26.0) (2024-03-05) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-cli: 0.26.0</summary> ## [0.26.0](AztecProtocol/aztec-packages@aztec-cli-v0.25.0...aztec-cli-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * Use new deployment flow in ContractDeployer ([#4497](AztecProtocol/aztec-packages#4497)) * move noir out of yarn-project ([#4479](AztecProtocol/aztec-packages#4479)) * note type ids ([#4500](AztecProtocol/aztec-packages#4500)) * Include contract class id in deployment info ([#4223](AztecProtocol/aztec-packages#4223)) * aztec binary ([#3927](AztecProtocol/aztec-packages#3927)) ### Features * **avm-transpiler:** Brillig to AVM transpiler ([#4227](AztecProtocol/aztec-packages#4227)) ([c366c6e](AztecProtocol/aztec-packages@c366c6e)) * Aztec binary ([#3927](AztecProtocol/aztec-packages#3927)) ([12356d9](AztecProtocol/aztec-packages@12356d9)) * Aztec.js API for registering a contract class ([#4469](AztecProtocol/aztec-packages#4469)) ([d566c74](AztecProtocol/aztec-packages@d566c74)) * Include contract class id in deployment info ([#4223](AztecProtocol/aztec-packages#4223)) ([0ed4126](AztecProtocol/aztec-packages@0ed4126)), closes [#4054](AztecProtocol/aztec-packages#4054) * Moving the unbox option to npx command ([#4718](AztecProtocol/aztec-packages#4718)) ([4c3bb92](AztecProtocol/aztec-packages@4c3bb92)) * Note type ids ([#4500](AztecProtocol/aztec-packages#4500)) ([e1da2fd](AztecProtocol/aztec-packages@e1da2fd)) * Parallel native/wasm bb builds. Better messaging around using ci cache. ([#4766](AztecProtocol/aztec-packages#4766)) ([a924e55](AztecProtocol/aztec-packages@a924e55)) * Use new deployment flow in ContractDeployer ([#4497](AztecProtocol/aztec-packages#4497)) ([0702dc6](AztecProtocol/aztec-packages@0702dc6)) ### Bug Fixes * Add new oracle contract to devnet in CI ([#4687](AztecProtocol/aztec-packages#4687)) ([920fa10](AztecProtocol/aztec-packages@920fa10)) * Load contract artifact from json ([#4352](AztecProtocol/aztec-packages#4352)) ([47a0a79](AztecProtocol/aztec-packages@47a0a79)) ### Miscellaneous * **docs:** Fix a few links to docs ([#4260](AztecProtocol/aztec-packages#4260)) ([1c8ea49](AztecProtocol/aztec-packages@1c8ea49)) * Move noir out of yarn-project ([#4479](AztecProtocol/aztec-packages#4479)) ([1fe674b](AztecProtocol/aztec-packages@1fe674b)), closes [#4107](AztecProtocol/aztec-packages#4107) * Remove stubbed docs ([#4196](AztecProtocol/aztec-packages#4196)) ([25a4bc4](AztecProtocol/aztec-packages@25a4bc4)) * Squash yp ypb + other build improvements. ([#4901](AztecProtocol/aztec-packages#4901)) ([be5855c](AztecProtocol/aztec-packages@be5855c)) * Updating viem ([#4783](AztecProtocol/aztec-packages#4783)) ([23bc26a](AztecProtocol/aztec-packages@23bc26a)) </details> <details><summary>aztec-packages: 0.26.0</summary> ## [0.26.0](AztecProtocol/aztec-packages@aztec-packages-v0.25.0...aztec-packages-v0.26.0) (2024-03-05) ### ⚠ BREAKING CHANGES * Internal as a macro ([#4898](AztecProtocol/aztec-packages#4898)) ### Features * Add init check by default to public fns ([#4897](AztecProtocol/aztec-packages#4897)) ([4550f25](AztecProtocol/aztec-packages@4550f25)) * Enable public constructor functions ([#4896](AztecProtocol/aztec-packages#4896)) ([7b06895](AztecProtocol/aztec-packages@7b06895)) * Internal as a macro ([#4898](AztecProtocol/aztec-packages#4898)) ([73d640a](AztecProtocol/aztec-packages@73d640a)) * We no longer update version packages via scripts ([#4962](AztecProtocol/aztec-packages#4962)) ([31d470b](AztecProtocol/aztec-packages@31d470b)) ### Miscellaneous * Disable failing test temporarily ([ec61974](AztecProtocol/aztec-packages@ec61974)) * Fixed call nesting, tests and docs ([#4932](AztecProtocol/aztec-packages#4932)) ([bd5c879](AztecProtocol/aztec-packages@bd5c879)) * Specify packages individually for release-please ([#4960](AztecProtocol/aztec-packages#4960)) ([dddc35f](AztecProtocol/aztec-packages@dddc35f)) * Sync noir repo ([#4947](AztecProtocol/aztec-packages#4947)) ([7ff9b71](AztecProtocol/aztec-packages@7ff9b71)) * Unused vars cleanup + updated TODOs ([#4883](AztecProtocol/aztec-packages#4883)) ([3747619](AztecProtocol/aztec-packages@3747619)) * Update escrow to use PrivateImmutable ([#4942](AztecProtocol/aztec-packages#4942)) ([245d801](AztecProtocol/aztec-packages@245d801)) </details> <details><summary>barretenberg: 0.26.0</summary> ## [0.26.0](AztecProtocol/aztec-packages@barretenberg-v0.25.0...barretenberg-v0.26.0) (2024-03-05) ### Miscellaneous * **barretenberg:** Synchronize aztec-packages versions </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Adds the init check by default to all public functions, unless they are flagged as
noinitcheck
.This should break all contracts that call a public function from their constructor. Let's see...