cartesi / rollups

Cartesi Rollups
30 stars 12 forks source link

Improve `genScript.ts` #200

Closed guidanoli closed 1 year ago

guidanoli commented 1 year ago

Closes #197

guidanoli commented 1 year ago

:test_tube: Benchmark

I've run update-proofs.sh before and after the change. Here are the results (on my x86 machine running Linux).

That's a huge gain right there. We achieve this level of speedup by not assuming a certain step will take up to $n$ seconds on all machines. Instead, we check whether we can move on to the next step as quick as possible.

ZzzzHui commented 1 year ago

On my MacBook, the run time decreases from 2'24" to 1'32". Well done!