-
Notifications
You must be signed in to change notification settings - Fork 310
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: Sync from noir #6801
feat: Sync from noir #6801
Conversation
…n arguments (noir-lang/noir#5142) fix(experimental elaborator): Fix frontend tests when `--use-elaborator` flag is specified (noir-lang/noir#5145) chore: Remove unused `new_variables` argument from `resolve_type_inner` (noir-lang/noir#5148) chore: Remove hir to ast pass (noir-lang/noir#5147) fix(experimental elaborator): Fix definition kind of globals and tuple patterns with `--use-elaborator` flag (noir-lang/noir#5139) chore: default to using bn254 in `noirc_frontend` (noir-lang/noir#5144) fix(experimental elaborator): Fix `impl Trait` when `--use-elaborator` is selected (noir-lang/noir#5138) fix: wrapping in signed division (noir-lang/noir#5134) chore(ci): don't raise MSRV issue if workflow cancelled (noir-lang/noir#5143) fix: use predicate for curve operations (noir-lang/noir#5076) fix(experimental elaborator): Fix global values used in the elaborator (noir-lang/noir#5135) fix(experimental elaborator): Clear generics after elaborating type aliases (noir-lang/noir#5136) fix(frontend): Resolve object types from method calls a single time (noir-lang/noir#5131) feat: Separate runtimes of SSA functions before inlining (noir-lang/noir#5121) chore: run all test programs in brillig as well as ACIR (noir-lang/noir#5128)
Changes to circuit sizes
🧾 Summary (100% most significant diffs)
Full diff report 👇
|
@guipublic fyi regression_5045 is failing to prove/verify in bb. |
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. Proof generationEach column represents the number of threads used in proof generation.
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 8 txs.
Circuits statsStats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.
Stats on running time collected for app circuits
Tree insertion statsThe duration to insert a fixed batch of leaves into each tree type.
MiscellaneousTransaction sizes based on how many contract classes are registered in the tx.
Transaction size based on fee payment method | Metric | | |
@@ -137,7 +137,7 @@ describe('e2e_fees private_payment', () => { | |||
* TODO(6583): update this comment properly now that public execution consumes gas | |||
*/ | |||
|
|||
expect(tx.transactionFee).toEqual(200036352n); | |||
expect(tx.transactionFee).toEqual(200032492n); |
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.
are these exact fee asserts a good idea?
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.
no
Automated pull of development from the noir programming language, a dependency of Aztec.
BEGIN_COMMIT_OVERRIDE
feat: place return value witnesses directly after function arguments (noir-lang/noir#5142)
fix(experimental elaborator): Fix frontend tests when
--use-elaborator
flag is specified (noir-lang/noir#5145)chore: Remove unused
new_variables
argument fromresolve_type_inner
(noir-lang/noir#5148)chore: Remove hir to ast pass (noir-lang/noir#5147)
fix(experimental elaborator): Fix definition kind of globals and tuple patterns with
--use-elaborator
flag (noir-lang/noir#5139)chore: default to using bn254 in
noirc_frontend
(noir-lang/noir#5144)fix(experimental elaborator): Fix
impl Trait
when--use-elaborator
is selected (noir-lang/noir#5138)fix: wrapping in signed division (noir-lang/noir#5134)
chore(ci): don't raise MSRV issue if workflow cancelled (noir-lang/noir#5143)
fix: use predicate for curve operations (noir-lang/noir#5076)
fix(experimental elaborator): Fix global values used in the elaborator (noir-lang/noir#5135)
fix(experimental elaborator): Clear generics after elaborating type aliases (noir-lang/noir#5136)
fix(frontend): Resolve object types from method calls a single time (noir-lang/noir#5131)
feat: Separate runtimes of SSA functions before inlining (noir-lang/noir#5121)
chore: run all test programs in brillig as well as ACIR (noir-lang/noir#5128)
END_COMMIT_OVERRIDE