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.

Sign up for GitHub

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

Jump to bottom

End-to-end constraint count tool for transactions #9299

Closed
saleel opened this issue Oct 21, 2024 · 2 comments
Closed

End-to-end constraint count tool for transactions #9299

saleel opened this issue Oct 21, 2024 · 2 comments
Assignees
@saleel
Milestone
TestNet

Comments

@saleel
Copy link
Contributor

saleel commented Oct 21, 2024

TBD
@saleel saleel added this to the TestNet milestone Oct 21, 2024
@saleel saleel self-assigned this Oct 21, 2024
@critesjosh
Copy link
Contributor

related: AztecProtocol/dev-rel#434

@github-project-automation github-project-automation bot moved this to Todo in A3 Oct 28, 2024
@saleel saleel mentioned this issue Oct 31, 2024
Merged
saleel added a commit that referenced this issue Nov 4, 2024
@saleel
feat: add gate count profiling for transactions (#9632)
582398f 
Part of #9299 

- Added `computeGateCountForCircuit` to `bb/execute.s` and use it in
`kernal_prover` to calculate gate-count per function (under a flag).
- Add `dryRun` to `kernal_prover` to skip ClientIVC proof generation
(useful for getting gate count without proof generation)
- Add `--profile` flag to cli-wallet `simulate` command to prints the
gate count per circuit, Example:
<img width="888" alt="Screenshot 2024-10-31 at 9 46 10 PM"
src="https://github.com/user-attachments/assets/4bbb062c-acdf-4889-aa6b-294c030b34d5">
<br/><br/>

- Not reusing `ContractFunctionInteraction.simulate` as it is difficult
to change its return type as its widely used.
- Can also add gas estimation to simulate command (instead of `send`) so
`simulate` does actual simulation of a transaction and display the full
data.
@critesjosh
Copy link
Contributor

closed by the linked PR

@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@saleel saleel

Labels
None yet
Projects
A3
Status: Done
Milestone
TestNet
Development

No branches or pull requests
2 participants
@saleel @critesjosh