Skip to content
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

Compare performance of the EdDSAPoseidon (and BabyJubjub) algorithm in JS and Rust in Node.js and browser #328

Open
cedoor opened this issue Sep 12, 2024 · 3 comments
Labels
documentation 📖 Improvements or additions to documentation good first issue Good for newcomers

Comments

@cedoor
Copy link
Member

cedoor commented Sep 12, 2024

Description

There is an implementation of EdDSA with Poseidon and BabyJubjub in TS/JS: https://github.com/privacy-scaling-explorations/zk-kit/tree/main/packages/eddsa-poseidon.

There is no EdDSA Poseidon (and BabyJubjub) implementation in Rust yet. Once the issue (privacy-scaling-explorations/zk-kit.rust#9) is completed, there will be a crate for it on zk-kit.rust.

The idea of this issue is to compare the performance of this algorithm written in TS/JS and Rust (compiled to wasm) in Node.js and browser.

Resources

Guide to convert the Rust implementation to wasm: https://rustwasm.github.io/docs/book/

Library to create benchmarks in Node.js and browser: https://github.com/tinylibs/tinybench

@cedoor cedoor added documentation 📖 Improvements or additions to documentation good first issue Good for newcomers labels Sep 12, 2024
@github-project-automation github-project-automation bot moved this to 📋 Backlog in ZK-Kit Sep 12, 2024
@Some-of-the-things
Copy link

So, just to clarify, this issue can't be picked up until privacy-scaling-explorations/zk-kit.rust#9 is complete? Perhaps it would be worth my compiling the documentation for the TS/JS part in the interim, if that is of interest?

@cedoor
Copy link
Member Author

cedoor commented Sep 23, 2024

Hey @Some-of-the-things, what documentation do you mean?

privacy-scaling-explorations/zk-kit.rust#9 is available as a good first issue again btw.

@Some-of-the-things
Copy link

@cedoor I'd assumed that the result will need documenting to share findings. Thanks for the heads up: I'll take a look at that one...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📖 Improvements or additions to documentation good first issue Good for newcomers
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants