EnergySystemsModellingLab / MUSE_OS

Welcome to the MUSE-OS repository
https://muse-os.readthedocs.io/en/latest/
GNU General Public License v3.0
22 stars 8 forks source link

Min production considered in supply #368

Open dalonsoa opened 5 days ago

dalonsoa commented 5 days ago

Description

Adds a minimum service constraint to the supply calculation, so it is consistent with the constraints used in the investment. This change only affects to the outputs of a couple of examples and tutorials - and only the MCAMetric-related outputs -, but none of the others, still producing the same outputs.

Fixes #335 , identified and further discussed in #321

Why there are two measures of supply instead of using the production calculated by the solver is another question and a matter for a different PR. After this PR is merged, both figures in #321 become the same, and equal to the one on the right.

Type of change

Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s.

Key checklist

Further checks

codecov[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.39%. Comparing base (5aeea97) to head (7b522aa).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## max_capacity #368 +/- ## ================================================ + Coverage 71.29% 71.39% +0.10% ================================================ Files 44 44 Lines 5880 5894 +14 Branches 1159 1162 +3 ================================================ + Hits 4192 4208 +16 + Misses 1366 1365 -1 + Partials 322 321 -1 ```

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