As evidenced by this failure: https://github.com/delvtech/hyperdrive/actions/runs/6359494239/job/17272622798?pr=601, our get_max_short function is very off (~60%) in certain edge cases. What may be going on is that we aren't accounting for matured positions in get_max_short. We should apply the updates that will be applied in _applyCheckpoint in the SDK to ensure that the starting share and bond reserves (as well as other accounting variables) are correctly accounted for.
As evidenced by this failure: https://github.com/delvtech/hyperdrive/actions/runs/6359494239/job/17272622798?pr=601, our
get_max_short
function is very off (~60%) in certain edge cases. What may be going on is that we aren't accounting for matured positions inget_max_short
. We should apply the updates that will be applied in_applyCheckpoint
in the SDK to ensure that the starting share and bond reserves (as well as other accounting variables) are correctly accounted for.