IAMconsortium / pyam

Analysis & visualization of energy & climate scenarios
https://pyam-iamc.readthedocs.io/
Apache License 2.0
226 stars 118 forks source link

Implement minor clean-up #601

Closed danielhuppmann closed 2 years ago

danielhuppmann commented 2 years ago

Description of PR

While working on the PR on supporting a mixed time domain, I noticed a few minor issues - putting them into this dedicated PR to avoid overloading an already quite complex PR...

codecov[bot] commented 2 years ago

Codecov Report

Merging #601 (f0a0c6b) into main (54c19ab) will not change coverage. The diff coverage is 83.3%.

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #601   +/-   ##
=====================================
  Coverage   94.2%   94.2%           
=====================================
  Files         50      50           
  Lines       5361    5361           
=====================================
  Hits        5052    5052           
  Misses       309     309           
Impacted Files Coverage Δ
tests/test_core.py 100.0% <ø> (ø)
pyam/timeseries.py 96.2% <66.6%> (ø)
pyam/core.py 94.3% <100.0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 54c19ab...f0a0c6b. Read the comment docs.