delvtech / agent0

Analysis & simulation repo for Delv
https://agent0.readthedocs.io/en/latest/
Apache License 2.0
52 stars 21 forks source link

Multi pool random hold #1644

Closed slundqui closed 3 months ago

slundqui commented 3 months ago

Fixing an issue with random hold policy when using the same policy on multiple pools.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.36%. Comparing base (fd4bf54) to head (61338aa). Report is 78 commits behind head on main.

Files with missing lines Patch % Lines
src/agent0/core/hyperdrive/policies/random_hold.py 93.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1644 +/- ## ========================================== + Coverage 81.25% 81.36% +0.11% ========================================== Files 98 98 Lines 7399 7406 +7 ========================================== + Hits 6012 6026 +14 + Misses 1387 1380 -7 ``` | [Flag](https://app.codecov.io/gh/delvtech/agent0/pull/1644/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/delvtech/agent0/pull/1644/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech) | `81.36% <94.11%> (+0.11%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.