Skip to content
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

refactor: remove EnqueuedCallSimulator #10015

Merged
merged 4 commits into from
Nov 18, 2024

Conversation

dbanks12
Copy link
Collaborator

@dbanks12 dbanks12 commented Nov 17, 2024

The EnqueuedCallSimulator crafted public inputs for the public kernel merge circuit at the enqueued call level. Now that public inputs are being crafted per public TX and the public kernel merge isn't used, this module can be removed.

Copy link
Collaborator Author

dbanks12 commented Nov 17, 2024

@dbanks12 dbanks12 force-pushed the db/remove-enqueued-call-simulator branch 2 times, most recently from 0dc52fb to fee500b Compare November 17, 2024 18:55
@dbanks12 dbanks12 added the e2e-all CI: Enables this CI job. label Nov 17, 2024
@dbanks12 dbanks12 force-pushed the db/remove-enqueued-call-simulator branch 2 times, most recently from 76c87c7 to 86eacaf Compare November 18, 2024 00:43
Copy link
Collaborator

@LeilaWang LeilaWang left a comment

Choose a reason for hiding this comment

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

Yay!

Base automatically changed from db/stop-calling-kernels to master November 18, 2024 12:56
@dbanks12 dbanks12 force-pushed the db/remove-enqueued-call-simulator branch from 86eacaf to 921db05 Compare November 18, 2024 14:49
@dbanks12 dbanks12 merged commit aa949ef into master Nov 18, 2024
101 checks passed
@dbanks12 dbanks12 deleted the db/remove-enqueued-call-simulator branch November 18, 2024 16:08
TomAFrench added a commit that referenced this pull request Nov 18, 2024
* master: (281 commits)
  fix: don't take down runners with faulty runner check (#10019)
  feat(docs): add transaction profiler docs (#9932)
  chore: hotfix runner wait (#10018)
  refactor: remove EnqueuedCallSimulator (#10015)
  refactor: stop calling public kernels (#9971)
  git subrepo push --branch=master noir-projects/aztec-nr
  git_subrepo.sh: Fix parent in .gitrepo file. [skip ci]
  chore: replace relative paths to noir-protocol-circuits
  git subrepo push --branch=master barretenberg
  chore: drop info to verbose in sequencer hot loop (#9983)
  refactor: Trace structure is an object (#10003)
  refactor: enqueued calls processor -> public tx simulator (#9919)
  chore: World state tech debt cleanup 1 (#9561)
  chore(ci): run noir tests in parallel to building e2e tests (#9977)
  Revert "chore: lower throughput of ebs disks" (#9996)
  feat: new proving broker implementation (#9400)
  chore: replace `to_radix` directive with brillig (#9970)
  chore: disable failing 48validator kind test (#9920)
  test: prove one epoch in kind (#9886)
  fix: formatting (#9979)
  ...
TomAFrench added a commit that referenced this pull request Nov 19, 2024
* master: (67 commits)
  chore: Fix bad merge on AztecLMDBStore initializer
  feat: add persisted database of proving jobs (#9942)
  chore: Clean up data configuration (#9973)
  chore: remove public kernels (#10027)
  chore: misc cleanup, docs and renaming (#9968)
  feat: IPA Accumulator in Builder (#9846)
  chore(docs): Updates to token contract (#9954)
  test(avm): minor benchmarking (#9869)
  chore(ci): run `l1-contracts` CI in parallel with `build` step (#10024)
  chore: build acir test programs in parallel to e2e build step (#9988)
  chore: pull out `array_set` pass changes (#9993)
  feat(avm): ephemeral avm tree (#9798)
  fix: don't take down runners with faulty runner check (#10019)
  feat(docs): add transaction profiler docs (#9932)
  chore: hotfix runner wait (#10018)
  refactor: remove EnqueuedCallSimulator (#10015)
  refactor: stop calling public kernels (#9971)
  git subrepo push --branch=master noir-projects/aztec-nr
  git_subrepo.sh: Fix parent in .gitrepo file. [skip ci]
  chore: replace relative paths to noir-protocol-circuits
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e-all CI: Enables this CI job.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants