-
Notifications
You must be signed in to change notification settings - Fork 542
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 methods to get info about ConstraintSystem #19
Comments
num_constraints
method to ConstraintSystem
Traitnum_constraints
method to ConstraintSystem trait
num_constraints
method to ConstraintSystem trait
I've created a test framework for simulating synthesis and getting statistics like this. The bellman API allows it to be implemented independently from this library: https://github.com/zcash-hackworks/sapling-crypto/tree/master/src/circuit/test The kinds of checks you're mentioning can be done using this, it even has constraint system hashing for integrity checking (useful for refactoring circuit code without intending to change its contents) and some printing of the constraint system and other stuff. |
I've seen (and used!) your |
Hm, I see what you mean. Maybe we can do this, but I have another idea. I also wanted to attach something to the API which allowed you to pass in an abstract I suppose that could be hooked up to get info about the constraint system also! |
That's a good idea, it sounds more general-purpose and informative! Let me know if I can help with implementing it. |
Add interface and computation of Sapling note nullifier
It would be useful to add
num_constraints
,num_inputs
andnum_aux
methods to theConstraintSystem
trait to get a quick sense of its cost. I think these methods are general enough to apply to all implementors ofConstraintSystem
. If you'd like I can send in a pull request for this as well.The text was updated successfully, but these errors were encountered: