delvtech / hyperdrive-rs

Rust SDK for the Hyperdrive AMM.

https://docs.rs/crate/hyperdrive-math/latest

Apache License 2.0

3 stars 0 forks source link

fix open long delta calculations and add tests #178

Closed dpaiton closed 4 months ago

dpaiton commented 4 months ago

Resolved Issues

working towards https://github.com/delvtech/hyperdrive-rs/issues/171 and https://github.com/delvtech/hyperdrive-rs/issues/21

Description

The code for calculating pool deltas & state after opening a long did not correctly account for the gov fee's impact on the share reserves. I fixed this and added a test to demonstrate the failure
I added a similar test on the short side, which almost passes with equality (seems there is a rounding error)
I renamed some functions, cleaned up some comments, fixed some typos