w3f / ring-proof

ring-vrf ring proof v2.5
Apache License 2.0
14 stars 4 forks source link

Powers of two multiples gadget implemented for Twisted Edwards Curves #36

Open drskalman opened 3 weeks ago

drskalman commented 3 weeks ago

Closes #35

davxy commented 1 week ago

@drskalman Is this identical to https://github.com/davxy/ring-proof/pull/5? If so, I propose moving forward with it, integrating it quickly so it can be used in JAM, and then submitting a PR here from my fork. I say so just to work twice to keep keep the PRs in sync.

drskalman commented 2 days ago

@drskalman Is this identical to davxy#5?

Basically; I just made the mistake of pushing https://github.com/w3f/ring-proof/pull/36/commits/7a1f3ff7d8589083fd71495e90b853cb9e78e9b5 to your branch, which I didn't mean to. But i don't see any harm in including an extra gadgedt in your fork. It is going to be used in verifying VRF in the SNARK for Verifiable.