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

chore(docs): updating the solidity contract how-to guide #6804

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

signorecello
Copy link
Contributor

There are some nits around wording and all, other than that the only change I made was moving everything to UltraHonk. Still unsure if this is the best solution, on one hand UltraHonk is the fastest, on another hand it's the most expensive...

closes #6761

@signorecello signorecello requested a review from a team December 13, 2024 11:25
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 13, 2024
Copy link
Contributor

Peak Memory Sample

Program Peak Memory
keccak256 79.14M
workspace 122.09M
regression_4709 295.99M
ram_blowup_regression 2.44G
private-kernel-tail 210.43M
private-kernel-reset 862.22M
private-kernel-inner 307.94M
parity-root 175.78M

Copy link
Contributor

Compilation Sample

Program Compilation Time %
sha256_regression 0m1.541s -10%
regression_4709 0m0.782s -3%
ram_blowup_regression 0m17.214s 0%
private-kernel-tail 0m1.352s -12%
private-kernel-reset 0m8.831s 1%
private-kernel-inner 0m2.510s -8%
parity-root 0m0.968s 3%
noir-contracts 2m46.869s -2%

Copy link
Contributor

Copy link
Contributor

FYI @noir-lang/developerrelations on Noir doc changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Solidity verifier how-to docs to 1.0.0-beta.0
1 participant