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

Add base64 example for gate counts #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jzaki
Copy link
Contributor

@jzaki jzaki commented Nov 1, 2024

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*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

- 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`
Copy link
Collaborator

Choose a reason for hiding this comment

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

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?

Copy link
Collaborator

@critesjosh critesjosh left a comment

Choose a reason for hiding this comment

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

What do you think about making the example more practical, in the sense that it would be something that someone would want to do in real life?

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

Successfully merging this pull request may close these issues.

Add base64 example using lib
2 participants