support computing proof with n arguments #89
Conversation
Signed-off-by: Antoine Toulme <[email protected]>
| public void testCallLibraryWithManyElements() { | ||
| Bytes input = Bytes.fromHexString("0x0cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec84"); | ||
| List<byte[]> params = new ArrayList<>(); | ||
| for (int i = 0 ; i < 128; i++) { |
There was a problem hiding this comment.
hey @atoulme i've been looking into the ipa-multipoint wrapper, and have one question. Where are these numbers pulled from? I mean 0x0cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec840cfe3041fb6512c87922e2146c8308b372f3bf967f889e69ad116ce7c7ec84 for example
Also, should we make the for loop go up to 256 values?
There was a problem hiding this comment.
It’s a test. I picked up this number, ran it with go impl, then used it here to create a test.
There was a problem hiding this comment.
@atoulme thanks, makes sense. Btw did you mean you ran it with rust impl?
There was a problem hiding this comment.
No with the go impl. Though rust should match
Signed-off-by: Antoine Toulme [email protected]
Instead of using 4 arguments, allow to use n arguments for the commit method.