curveresearch / curvesim

Simulates Curve Finance pools
MIT License
156 stars 32 forks source link

`StateLog` and `Trader` should be stateless #288

Open chanhosuh opened 11 months ago

chanhosuh commented 11 months ago

Right now, StateLog and Trader instances have a pool attribute that may get mutated from elsewhere. There is no need for state however, so their methods should be updated with a pool arg instead.