zerokit icon indicating copy to clipboard operation
zerokit copied to clipboard

Published 20 hours ago • verified publisher icon vacp2p

Enable parallel execution in rln-wasm

Open rahulghangas opened this issue 2 years ago • 3 comments

  • [ ] Closes #55

rahulghangas avatar Feb 01 '23 19:02 rahulghangas

Is this PR still draft or is it ready for review?

oskarth avatar Feb 08 '23 06:02 oskarth

rln-wasm tests are failing, so I guess is still in draft? Furthermore, I believe these changes has to be fully tested with all submodules that use the "parallel" feature (rln, so groth16, arkworks, etc.) and the output WASM should be tested with js-rln too to see if it is working as expected @richard-ramos It is unclear to me which browsers currently support rayon, so this is something we should take into account too (e.g. when distributing or setting defaults for rln-wasm).

s1fr0 avatar Feb 08 '23 09:02 s1fr0

Still a draft, it compiles but tests get deadlocked for some reason, I think because the pool doesn't get initialized on the client side. Will figure this out today

rahulghangas avatar Feb 08 '23 11:02 rahulghangas