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: bump circom to 2.1.6 #105

Closed
wants to merge 1 commit into from
Closed

Conversation

lonerapier
Copy link
Collaborator

closes #92

There are no constraint changes from previous version, but the actual value doesn't match with the one provided in README.

non-linear constraints: 2801097
linear constraints: 0
public inputs: 1
public outputs: 9
private inputs: 2631
private outputs: 0
wires: 2704830
labels: 11813196

@Divide-By-0
Copy link
Member

Are there no bumps that need to be made in package.json to the circom or snarkjs packages?

@lonerapier
Copy link
Collaborator Author

circom is installed separately, right.

Should we use the latest version of snarkjs or update the fork to latest upstream?

@Divide-By-0
Copy link
Member

We still need to use the forks due to the chunking. If you can update those forks to latest versions of snark js and have tests to ensure they still work with chunking, that would be ideal!

@Divide-By-0
Copy link
Member

Done in #185

@Divide-By-0 Divide-By-0 closed this Apr 1, 2024
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.

Circom Bump to 2.1.6
2 participants