-
Notifications
You must be signed in to change notification settings - Fork 249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for ristretto255 / sr25519 curve #819
Comments
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. |
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:
***@***.***>
|
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. |
https://github.com/nikkolasg/ark-blst looks relevant |
Feature request 👍
The text was updated successfully, but these errors were encountered: