iiasa / ixmp

The ix modeling platform for integrated and cross-cutting scenario analysis
https://docs.messageix.org/ixmp
Apache License 2.0
38 stars 112 forks source link

Improve ixmp.model internals #376

Closed khaeru closed 4 years ago

khaeru commented 4 years ago

How to review

Note that the CI checks all pass.

PR checklist

codecov[bot] commented 4 years ago

Codecov Report

Merging #376 into master will increase coverage by 0.0%. The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #376   +/-   ##
======================================
  Coverage    96.8%   96.8%           
======================================
  Files          46      46           
  Lines        5253    5281   +28     
======================================
+ Hits         5086    5114   +28     
  Misses        167     167           
Impacted Files Coverage Δ
ixmp/backend/base.py 98.8% <ø> (ø)
ixmp/core.py 94.8% <ø> (ø)
ixmp/model/base.py 100.0% <100.0%> (ø)
ixmp/model/gams.py 100.0% <100.0%> (ø)
ixmp/tests/core/test_scenario.py 100.0% <100.0%> (ø)
ixmp/tests/test_model.py 100.0% <100.0%> (ø)
ixmp/utils.py 96.2% <100.0%> (+0.1%) :arrow_up: