ApeWorX / ape-foundry

Foundry network provider for the Ape Framework
https://www.apeworx.io/
Apache License 2.0
33 stars 13 forks source link

perf: optimize getting virtual machine errors #115

Closed antazoey closed 2 months ago

antazoey commented 2 months ago

What I did

Before:

Wall time: 554 ms

Then with https://github.com/ApeWorX/ape/pull/2211

Wall time: 450 ms

Then with this PR and not https://github.com/ApeWorX/ape/pull/2211

Wall time: 273 ms

Then with this PR AND https://github.com/ApeWorX/ape/pull/2211

Wall time: 245 ms

How I did it

I think the main thing was not defining such a large method within another method but also some condition combines probably helped.

How to verify it

should work like it did before

Checklist

antazoey commented 2 months ago

Interesting

Screenshot 2024-08-16 at 08 39 09