barustenberg icon indicating copy to clipboard operation
barustenberg copied to clipboard

fix(pederson): Add correct values to pederson hash and pederson commit tests

Open PatStiles opened this issue 1 year ago • 0 comments

NB: Feature requests will only be considered if they solve a pain or present a useful refactoring of the code.

Summary

Problem

The pederson commitment tests are currently failing due to testing agains the hard-coded generator values from barretenburg. Due to limitations in Arkworks we use a slightly different canonical method for creating generators by hashing to a curve.

Solution

To fix these tests trace over the hash-to-curve function and manually compute the resulting pederson hash values for Fq::one() from the generators.

PatStiles avatar Nov 06 '23 01:11 PatStiles