You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
case B) In case that we don't use randomness for the commitments, we don't need r_w at all and can just be removed. (which, seeing the pedersen::CommitmentEngine, it does not use any randomness internally for the commitment.
So, if we go with 'case A', we should return r_w at the end of the method (together with the witness) and probably update the Pedersen implementation to support using randomness, and if we go with 'case B' we don't need to define _r_w.
As @arnaucube correctly stated above, we should take a decision and either remove completelly the usage of randomness for the Pedersen Commitments, or, change the current impl of the Pedersen Commitments and actually include the randomness support for them.
The text was updated successfully, but these errors were encountered:
Originally posted by @arnaucube in #41 (comment)
As @arnaucube correctly stated above, we should take a decision and either remove completelly the usage of randomness for the Pedersen Commitments, or, change the current impl of the Pedersen Commitments and actually include the randomness support for them.
The text was updated successfully, but these errors were encountered: