-
Notifications
You must be signed in to change notification settings - Fork 15
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
Batch verify not working #66
Comments
@saideng the test is seems to be working for me. What's the error you're getting? A thing to note, the test will require more than 9-10G of memory, does your machine running the test satisfy that? |
The test case with one signature works, but the test cases with two and three signatures do not work for me. I am using MacBook Pro with 16gb memory. |
@saideng By any chance is it M1 (Macbook pro)? |
@saideng this is likely a resource issue, the test case with 2 and 3 signatures would require more memory (about 20 and 30G). You can add swap space on your mac and use that to compile the circuits and compute corresponding witnesses. If you're getting a specific error, could you post that so that we can investigate further |
@jinankjain Yes, it is M1. @rahulghangas I tried to compile a 3 signature batch verify and got # of Constraints: 10953937. I think this is not scalable unless using recursive construction. Have you tried other tools or libraries for recursive SNARK? |
Thanks for the repo, and it is very cool!
I wanted to try out batch verify but the tests failed. Does it work? Do you have benchmarks of the batch verify circuit?
The text was updated successfully, but these errors were encountered: