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 Alternate Noir Versions to our DB #70

Open
katiemckeon opened this issue Feb 9, 2024 · 1 comment
Open

Add Alternate Noir Versions to our DB #70

katiemckeon opened this issue Feb 9, 2024 · 1 comment
Assignees

Comments

@katiemckeon
Copy link
Contributor

With broader version support for Noir Circuits, we need to update our sample circuits to include each possible supported version. (Bonus points for circuits with features not supported by previous versions!)

@katiemckeon katiemckeon self-assigned this Feb 12, 2024
katiemckeon added a commit that referenced this issue Feb 12, 2024
This PR updates all of our current Noir circuits (both in
`circuit_database/` and in `circuit_tutorials/`) to include the new
field from the Sindri manifest: `noirVersion`. This field allows users
to specify whether we should apply the Noir prover version [0.17.0,
0.18.0, 0.19.4, 0.22.0, 0.23.0]. This is the first of two PR's that
patches our existing circuits, which have been assuming the 0.17.0
prover version. The next PR introduces more circuits of other versions -
see #70

## Reviewer Testing:
- [x] For each compressed circuit in `circuit_database/noir`, submit to
the frontend (stage server) and ensure compilation is successful:
  - [ ] not_equal.tar.gz
  - [ ] not_equal.zip
  - [x] decision_tree.tar.gz
  - [x] decision_tree.zip
  - [x] billion_zk_vote.tar.gz
  - [x] billion_zk_vote.zip
- [x] Ensure the `sindri.json` file at
`circuit_tutorials/noir/pagerank/circuits` has a field for the
`noirVersion` and that it is set to `0.17.0`
@KPreisner
Copy link
Member

Is this ticket still needed?

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

No branches or pull requests

2 participants