search

assume-framework / assume

ASSUME - Agent-based Simulation for Studying and Understanding Market Evolution
https://assume.readthedocs.io
20 stars 5 forks source link

Remove empty bid as method of bidding strategy #293

Closed nick-harder closed 5 months ago

nick-harder commented 5 months ago

Removes the clearing of empty bids from unit operator into the bidding strategies themselves. This allows for the user to control this behavior and keep bids with volume of 0 if needed (such as for redispatch market)

codecov[bot] commented 5 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (935b69d) 83.12% compared to head (7dfc4ca) 83.09%.

Files Patch % Lines
assume/strategies/extended.py 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #293 +/- ## ========================================== - Coverage 83.12% 83.09% -0.04% ========================================== Files 40 40 Lines 4564 4585 +21 ========================================== + Hits 3794 3810 +16 - Misses 770 775 +5 ``` | [Flag](https://app.codecov.io/gh/assume-framework/assume/pull/293/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | Coverage Δ | | |---|---|---| | [pytest](https://app.codecov.io/gh/assume-framework/assume/pull/293/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | `83.09% <96.15%> (-0.04%)` | :arrow_down: | 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=assume-framework#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.