assume-framework / assume

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

improve messaging with typedDicts #245

Closed maurerle closed 10 months ago

maurerle commented 10 months ago

fixes an error when adjust marginal_costs for multiple markets use market_id consistently receive registration feedback from market at start of simulation

codecov[bot] commented 10 months ago

Codecov Report

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

Comparison is base (5834480) 78.13% compared to head (6a59d6c) 78.28%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #245 +/- ## ========================================== + Coverage 78.13% 78.28% +0.15% ========================================== Files 40 40 Lines 4198 4242 +44 ========================================== + Hits 3280 3321 +41 - Misses 918 921 +3 ``` | [Flag](https://app.codecov.io/gh/assume-framework/assume/pull/245/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/245/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | `78.28% <92.95%> (+0.15%)` | :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=assume-framework#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | Coverage Δ | | |---|---|---| | [assume/common/base.py](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#diff-YXNzdW1lL2NvbW1vbi9iYXNlLnB5) | `78.92% <100.00%> (ø)` | | | [assume/common/market\_objects.py](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#diff-YXNzdW1lL2NvbW1vbi9tYXJrZXRfb2JqZWN0cy5weQ==) | `100.00% <100.00%> (ø)` | | | [assume/common/outputs.py](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#diff-YXNzdW1lL2NvbW1vbi9vdXRwdXRzLnB5) | `50.25% <75.00%> (+0.25%)` | :arrow_up: | | [assume/common/units\_operator.py](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#diff-YXNzdW1lL2NvbW1vbi91bml0c19vcGVyYXRvci5weQ==) | `90.54% <89.47%> (-0.46%)` | :arrow_down: | | [assume/markets/base\_market.py](https://app.codecov.io/gh/assume-framework/assume/pull/245?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#diff-YXNzdW1lL21hcmtldHMvYmFzZV9tYXJrZXQucHk=) | `87.50% <90.90%> (-0.14%)` | :arrow_down: |

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