Tricrypto `remove_liquidity_one_coin`

curveresearch / curvesim

Simulates Curve Finance pools

MIT License

155 stars 32 forks source link

Tricrypto `remove_liquidity_one_coin` #218

Closed chanhosuh closed 1 year ago

chanhosuh commented 1 year ago

Currently CurveCryptoPool.remove_liquidity_one_coin does not handle 3 coins, only 2. Update the implementation in a clean manner to handle this. Similar test as for 2 coins should work. Reference implementation is the Tricrypto-NG vyper fixture.

Note: This is blocked by calc_withdraw_one_coin.

allt0ld commented 1 year ago

Tricrypto claims admin fees before withdrawal (L772), but cryptopool doesn't. This difference links this issue to #231