Slowness to generate proofs with recent zerokit versions

[richard-ramos]

As described in these comments:

• feat: use identity credentials, and expose hash, bulk insert and delete members functions #58 (comment)
• feat: use identity credentials, and expose hash, bulk insert and delete members functions #58 (comment)
• move to uncompressed versions vacp2p/zerokit#161 (comment)
  The time to generate a proof increased a lot going from ~3s to ~23s, with this commit vacp2p/zerokit@5eb98d4 introducing the slowness

[weboko]

My understanding is that in this issue we should:

• investigate why it happens
• mitigate on js side if possible

@richard-ramos is it correct?

[richard-ramos]

Indeed, but i'm not entirely sure we'll be able to mitigate the issue on JS side as it seems to have been introduced in zerokit

[rymnc]

@tyshko-rostyslav any idea why this is happening?

[tyshko-rostyslav]

If indeed this commit vacp2p/zerokit@5eb98d4 is the reason, it's quite odd, nothing special is happening there

[rymnc]

@tyshko-rostyslav is it possible the move from ark-circom 0.3.0 to 0.4.0 is the problem?

[tyshko-rostyslav]

There was a significant slowdown, when we changed versions, specifically a lot of operations became compressed. As you've seen in this thread, I tried uncompressed versions in ark-circom 0.4, which somewhat helped, but not to the desired effect

[rymnc]

right, I think it'd be more performant if js-rln used a similar approach to https://github.com/Rate-Limiting-Nullifier/rlnjs, maybe even use the package to avoid de-duped work. ofcourse, they use rln-v2, but an older commit will be your best bet

[weboko]

I need to look into how we can leverage https://github.com/Rate-Limiting-Nullifier/rlnjs

[weboko]

rlnjs was not selected to be used due to fact that we needed to maintain proxy in JS code for our RLN contract and be able to optimize code in our own way.

Ice boxing this task as in future we might have a light weight way of generating / verifying proofs. Potentially we can use parts of rlnjs for this to avoid using wasm version of zerokit.

@adklempner @waku-org/js-waku-developers