Support for ristretto255 / sr25519 curve

[drewstone]

Feature request 👍

[burdges]

https://github.com/arkworks-rs/algebra/tree/master/curves/ed25519

We should've a curve25519-dalek wrapper though, because curve25519-dalek is much faster than arkworks. Arkworks exploits multi-threading, which curve25519-dalek lacks, but if you system is pegged anyways then curve25519-dalek remains a better choice.

[drewstone]

It’s a nice to have here as well, especially with other libraries that leverage Arkworks as a base (certain threshold sig libraries).

…

On Thu, Apr 18, 2024 at 10:35 PM Jeff Burdges ***@***.***> wrote: https://github.com/arkworks-rs/algebra/tree/master/curves/ed25519 We should've a curve25519-dalek wrapper though, because curve25519-dalek is much faster than arkworks. Arkworks exploits multi-threading, which curve25519-dalek lacks, but if you system is pegged anyways then curve25519-dalek remains a better choice. — Reply to this email directly, view it on GitHub <#819 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADELLFZMM37OBFUDH3AEOSDY6CNJTAVCNFSM6AAAAABGN3WZSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRVG42DAMZVHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[burdges]

We do have ed25519 here, but the problem here is the lack of a curve25519-dalek wrapper.

A zk prover team might only care aobut their benchmarks on dedicated prover nodes. In the polkadot ecosystem, we do care about total CPU time on validators, also now collators after elastic scaling, so the multi-threaded impl here should be worse than cruve25519-dalek.