w3f / ring-proof

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

Optional ark-transcript and merlin #30

Closed davxy closed 1 month ago

davxy commented 1 month ago
burdges commented 1 month ago

Is there a reason to keep merlin?

davxy commented 1 month ago

No not really, I added it just because is a quite popular implementation for FS and provide a couple of "out of the box" optional alternatives.

But if you prefer I can remove it. Also, we can put ark-transcript in the default features if you like. Whatever :-)

burdges commented 1 month ago

I've no optinion, just curious. I didn't make transcript traits around ArkTranscript because doing so adds complexity downstream. I guess that's still the right choice, but never sure. ;)

swasilyev commented 1 month ago

We decided not to use merlin. Also not a fan of extra features.