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

Update ixmp.jar and dependencies #370

Closed fonfon closed 4 years ago

fonfon commented 4 years ago

Update the ixmp.jar and the java dependencies.

How to review

It'd be good to check that functionality related to the dependencies, like the r bindings or running on various platforms, works. The test_tutorials.py tests are failing - this has to be clarified before merging (thus the draft state). update With #371 being merged the r-tests pass now, thanks Paul!

PR checklist

codecov[bot] commented 4 years ago

Codecov Report

Merging #370 into master will not change coverage. The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #370   +/-   ##
======================================
  Coverage    96.8%   96.8%           
======================================
  Files          46      46           
  Lines        5253    5253           
======================================
  Hits         5086    5086           
  Misses        167     167           
Impacted Files Coverage Δ
ixmp/tests/core/test_meta.py 100.0% <100.0%> (ø)
khaeru commented 4 years ago

The test_tutorials.py tests are failing

I investigated and fixed this in #371. Please rebase this branch and confirm that checks pass.