-
Notifications
You must be signed in to change notification settings - Fork 9
Enable parallel execution for rln-wasm module #296
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
base: master
Are you sure you want to change the base?
Conversation
Benchmark for 0e27fa8Click to view benchmark
|
rln-wasm/Cargo.toml
Outdated
@@ -37,3 +42,4 @@ wasm-bindgen-futures = "0.4.50" | |||
default = ["console_error_panic_hook"] | |||
stateless = ["rln/stateless"] | |||
arkzkey = ["rln/arkzkey"] | |||
rayon-parallelism = ["wasm-bindgen-rayon"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's choose a shorter name, like we used to - parallel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll come up with a better feature name, but parallel
doesn't work because it enables some internal dependency features that aren't supposed to be active.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm, I got it, then it makes sense to get something else
Benchmark for 0e27fa8Click to view benchmark
|
Benchmark for 0e27fa8Click to view benchmark
|
Benchmark for 0431902Click to view benchmark
|
Benchmark for 0431902Click to view benchmark
|
Benchmark for 0431902Click to view benchmark
|
…ser-compatible witness calculators; fix Makefile.toml; thread pool works in HTML but not in wasm-bindgen-test
Benchmark for 3a210a3Click to view benchmark
|
Benchmark for 3a210a3Click to view benchmark
|
Benchmark for 3a210a3Click to view benchmark
|
…ed wasm-bindgen-rayon maintainer for help on this issue
Benchmark for ade70adClick to view benchmark
|
Benchmark for ade70adClick to view benchmark
|
Benchmark for 1650689Click to view benchmark
|
Benchmark for ade70adClick to view benchmark
|
Benchmark for 1650689Click to view benchmark
|
Benchmark for 1650689Click to view benchmark
|
Benchmark for 7160c42Click to view benchmark
|
Benchmark for 7160c42Click to view benchmark
|
Benchmark for 7160c42Click to view benchmark
|
Benchmark for 147e66cClick to view benchmark
|
Benchmark for b60ee07Click to view benchmark
|
Benchmark for 147e66cClick to view benchmark
|
Benchmark for 147e66cClick to view benchmark
|
Benchmark for 3aafbb6Click to view benchmark
|
Benchmark for 3aafbb6Click to view benchmark
|
Benchmark for 3aafbb6Click to view benchmark
|
…m-bindgen-test in a seperate branch
…hmark folder (forked into seperate branch) for the PR review
Benchmark for c18d140Click to view benchmark
|
Benchmark for c18d140Click to view benchmark
|
Benchmark for c18d140Click to view benchmark
|
Benchmark for 43f65f8Click to view benchmark
|
Benchmark for 43f65f8Click to view benchmark
|
Benchmark for 43f65f8Click to view benchmark
|
Benchmark for 16ab01fClick to view benchmark
|
Benchmark for 16ab01fClick to view benchmark
|
Benchmark for 16ab01fClick to view benchmark
|
…date and improve installdeps command, resolve issue related to enable nvm in current terminal session
Benchmark for 631621fClick to view benchmark
|
Benchmark for 631621fClick to view benchmark
|
Benchmark for 631621fClick to view benchmark
|
Changes
rln-wasm
with themultithread
feature flag.wasm-bindgen-rayon
(comment).rln-wasm
.make installdeps
which resolves the issue of NVM not enabling Node.js in the current terminal session..wasm
blob using thewasm-opt
tool from Binaryen.