IAMconsortium / pyam

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

Remove ixmp as optional dependency #688

Closed danielhuppmann closed 1 year ago

danielhuppmann commented 1 year ago

Please confirm that this PR has done the following:

Description of PR

This PR removes the ixmp packages as an optional dependency. The feature to read IAMC-style scenario data directly from an ixmp database instance was untested, does not support meta-indicators and, as far as I know, is not used by anyone.

The feature to query data from an IIASA Scenario Explorer database (pyam.read_iiasa()) is not affected by this PR!

The PR prepares for the release of ixmp 4.0.0, based on a discussion with @meksor and @phackstock today.

@gidden @OFR-IIASA @behnam-zakeri @byersiiasa, please shout if you are using this feature or know of anyone who does...

codecov[bot] commented 1 year ago

Codecov Report

Merging #688 (4adceb3) into main (d104630) will increase coverage by 0.1%. The diff coverage is n/a.

@@           Coverage Diff           @@
##            main    #688     +/-   ##
=======================================
+ Coverage   94.7%   94.8%   +0.1%     
=======================================
  Files         59      58      -1     
  Lines       5836    5814     -22     
=======================================
- Hits        5529    5517     -12     
+ Misses       307     297     -10     
Impacted Files Coverage Δ
pyam/__init__.py 67.5% <ø> (-0.9%) :arrow_down:
pyam/core.py 94.8% <ø> (+0.2%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

danielhuppmann commented 1 year ago

Will merge in one week (August 26) if I do not hear any objections...