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

Readme/constraint breakdown for the email wallet #100

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

whoisgautxm
Copy link
Collaborator

Fixes #11
This is the breakdown of all the non-linear constraints in all the email-wallet circuits
// main circuits

Claim.circom - constraints 921
Announcement.circom - constraints 459
Email_sender - constraints 1895054
Account_creation - constarinst 2102567

// utils

Account_salt.circom - constraints 258
Bytes2ints.circom - constraints 0
Digit2int.circom - constarints 0
Email_addr_commit.circom - constraints 465
Email_nullifier.circom - constraints 213
Hash_sign.circom - constraints 213
Hext2int.circom - constaints 80

//regex
Invitation_code_regex.circom - constaints 97
Invitation_code_with_prefix_regex.circom - constraints 105

// Total no of constraints - 3,999,432

@Divide-By-0
Copy link
Member

Divide-By-0 commented Aug 14, 2024

I wanted more of a breakdown -- how much are the SHAs and Regexes each? Youll have to deconstruct each regex with the correct arguments. Looking forward to updates.

Copy link
Member

@Divide-By-0 Divide-By-0 left a comment

Choose a reason for hiding this comment

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

Fix constraint breakdowns for regexes.

@Divide-By-0
Copy link
Member

Please remove breakdowns from the two readmes (only leave the one line for the constraint total), and put it in https://github.com/zkemail/docs/ instead and hyperlink to that.

@whoisgautxm
Copy link
Collaborator Author

Please remove breakdowns from the two readmes (only leave the one line for the constraint total), and put it in https://github.com/zkemail/docs/ instead and hyperlink to that.

Ohk , working on that

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.

Write a constraint breakdown for the wallet
2 participants