keep-starknet-strange / raito

Bitcoin ZK client written in Cairo.
https://raito.wtf
MIT License
40 stars 34 forks source link

optimise gas #174

Closed Jeanmichel7 closed 1 month ago

Jeanmichel7 commented 1 month ago

Code optimization according to #156 instructions.

Test Before After
full_169 46789012 45078120
full_757738 193334626 1930227282
light_150012 6328008 4065106
light_169 5754468 4041136
light_2015 5754468 4041136
light_209999 6328008 4065106
light_24834 5754468 4041136
light_32255 7899588 4796716
light_478557 6901548 4081086
light_481823 10193748 4756406
light_491406 7188318 4097866
light_57042 5754468 4049126
light_629999 7188318 4089076
light_709631 10847918 4835686
light_757738 7188318 4089076
light_757752 7188318 4089076
light_774627 7188318 4089076
light_839999 7188318 4089076
vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
raito ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 11, 2024 7:57pm
maciejka commented 1 month ago

@Jeanmichel7 are you going to move this forward?

Jeanmichel7 commented 1 month ago

@maciejka yes I'm just finishing my utreexo issue first

m-kus commented 1 month ago

Many things have changed in the codebase (and some changes are in pending PRs), it probably makes sense to close this one and make a new, which fixes the low hanging fruits (like < -> != etc, without significant changes in the implementation).