search

IAMconsortium / pyam

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

Fix pandas v1.4 breaks the IIASA database API #609 #621

Closed coroa closed 2 years ago

coroa commented 2 years ago

pd.read_json in pandas 1.4 expects to receive strings not bytes.

danielhuppmann commented 2 years ago

It seems that pandas v1.4 breaks a few more items, related to the filter implementation. If you have a minute to investigate what is going on...?

Note that I have a large PR #598, with a specific PR #614 moving the filter-implementation to a new (test) module. I hope to have that finished by end of next week, will then come back to this PR.

codecov[bot] commented 2 years ago

Codecov Report

Merging #621 (3ce0cb6) into main (21ada20) will not change coverage. The diff coverage is 100.0%.

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #621   +/-   ##
=====================================
  Coverage   94.3%   94.3%           
=====================================
  Files         54      54           
  Lines       5444    5444           
=====================================
  Hits        5138    5138           
  Misses       306     306
Impacted Files Coverage Δ
tests/test_cast_to_iamc.py 100.0% <ø> (ø)
pyam/core.py 94.6% <100.0%> (+<0.1%) :arrow_up:
pyam/iiasa.py 85.8% <100.0%> (ø)
tests/test_admin.py 95.0% <100.0%> (ø)
tests/test_feature_aggregate.py 98.8% <100.0%> (ø)
tests/test_feature_append_rename.py 100.0% <100.0%> (ø)
tests/test_feature_growth_rate.py 100.0% <100.0%> (ø)
tests/test_iiasa.py 92.6% <100.0%> (ø)
tests/test_io.py 100.0% <100.0%> (ø)
tests/test_plotting.py 100.0% <100.0%> (ø)
... and 3 more

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 21ada20...3ce0cb6. Read the comment docs.