calliope-project / calliope

A multi-scale energy systems modelling framework
https://www.callio.pe
Apache License 2.0
287 stars 93 forks source link

Update parameter names #494

Closed brynpickering closed 11 months ago

brynpickering commented 11 months ago

Fixes issue #451

Summary of changes in this pull request:

Reviewer checklist:

brynpickering commented 11 months ago

I expect some tests to fail as I originally built this off #492, so some tests expect logging outputs that look different.

Future PRs will build off the combination of this PR and #492.

codecov[bot] commented 11 months ago

Codecov Report

Merging #494 (37d7fb9) into main (cbed4c8) will decrease coverage by 2.77%. Report is 7 commits behind head on main. The diff coverage is 96.15%.

@@            Coverage Diff             @@
##             main     #494      +/-   ##
==========================================
- Coverage   94.98%   92.22%   -2.77%     
==========================================
  Files          31       32       +1     
  Lines        3890     4037     +147     
  Branches        0      959     +959     
==========================================
+ Hits         3695     3723      +28     
- Misses        195      198       +3     
- Partials        0      116     +116     
Files Coverage Δ
src/calliope/__init__.py 100.00% <ø> (ø)
src/calliope/_version.py 100.00% <ø> (ø)
src/calliope/backend/helper_functions.py 99.35% <ø> (ø)
src/calliope/backend/where_parser.py 98.55% <100.00%> (ø)
src/calliope/cli.py 81.67% <100.00%> (ø)
src/calliope/core/__init__.py 100.00% <ø> (ø)
src/calliope/core/attrdict.py 99.50% <ø> (ø)
src/calliope/core/io.py 91.75% <100.00%> (ø)
src/calliope/core/model.py 93.23% <100.00%> (ø)
src/calliope/core/util/generate_runs.py 87.09% <ø> (ø)
... and 22 more
brynpickering commented 11 months ago

I don't get the docs failure locally, so I'm hoping it's a random glitch

brynpickering commented 11 months ago

Error is caused by mamba ignoring ruamel.yaml < 0.18 and seeing it instead as ruamel.yaml <= 0.18. Pinned for extra security.