Add base64 example for gate counts #28
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
critesjosh
reviewed
Nov 6, 2024
| - Note there's an optional param that you can use to specify the name of the witness file generated | ||
| - Optional: `nargo execute -p Prover_SHORT.toml base64_example_SHORT.gz` | ||
| - If using barretenberg, check the gate count referring to the newly created program: | ||
| - `bb gates -b ../target/base64_example.json` |
There was a problem hiding this comment.
Since we suggest running this, do you think we should provide some context as to what the output of bb gates means? Or link to a place that explains it?
There was a problem hiding this comment.
Added a point for some context.
critesjosh
reviewed
Nov 6, 2024
jzaki
force-pushed
the
jz/base64_example
branch
from
8e7b198 to
7b21267
Compare
Added a point with some application context/idea. |
critesjosh
reviewed
Jan 30, 2025
|
|
||
| The lib covers several encode/decode tests, whereas this program has some extra code to facilitate benchmarking the gate counts of a proving backend. Comparisons can be made against other proving backends if desired. | ||
|
|
||
| Ordinarily base64 is used encode/decode binary data sent across the web (in html/css files), rather than the text strings shown here. So could be used to verify that a website sent a particular image by comparing to a local binary, with applications that could equate to digital watermarks. |
There was a problem hiding this comment.
Interesting idea. just curious if you have ideas about when someone might want to do this in practice?
critesjosh
approved these changes
Jan 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Problem*
For libs to remain backend agnostic, proving backend gate counts should exist outside of the lib. And generally for examples to use libs in some context.
Resolves #29
Summary*
A place to show bb gate counts of libs, first the base64 lib.
Additional Context
PR Checklist*
cargo fmton default settings.