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.

Sign up for GitHub

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

Jump to bottom

Don't publish circuit keys in npm package #264

Open
vimwitch opened this issue Dec 14, 2022 · 0 comments
Open

Don't publish circuit keys in npm package #264

vimwitch opened this issue Dec 14, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@vimwitch
Copy link
Member

The defaultProver included with @unirep/circuits uses a bundled set of keys for relatively small tree sizes. This means when we increase the tree sizes for production the behaviors can change, and we don't test for that.

Instead we can avoid bundling small circuit keys and instead build production ready keys that are retrieved from somewhere on the web.
@vimwitch vimwitch self-assigned this Jan 4, 2023
@vimwitch vimwitch removed their assignment Jan 31, 2023
@vimwitch vimwitch self-assigned this Feb 14, 2023
@vimwitch vimwitch removed their assignment Feb 28, 2023
@vimwitch vimwitch self-assigned this Mar 29, 2023
@vimwitch vimwitch removed their assignment Apr 26, 2023
@vivianjeng vivianjeng added the enhancement New feature or request label May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vimwitch @vivianjeng