boutproject / xBOUT

Collects BOUT++ data from parallelized simulations into xarray.
https://xbout.readthedocs.io/en/latest/
Apache License 2.0
22 stars 10 forks source link

Pass `combine_attrs` arguments for compatibility with xarray-0.18; bump to xarray-0.17.0, Python 3.7 #199

Closed johnomotani closed 3 years ago

johnomotani commented 3 years ago

Need to pass combine_attrs arguments explicitly in a couple of places, otherwise tests fail with the most recent release of xarray. The new arguments require bumping the minimum supported version of xarray to 0.17.0, which requires bumping the minimum Python version to 3.7.

codecov-commenter commented 3 years ago

Codecov Report

Merging #199 (becd461) into master (8f14cb2) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #199   +/-   ##
=======================================
  Coverage   74.61%   74.61%           
=======================================
  Files          14       14           
  Lines        2600     2600           
  Branches      617      617           
=======================================
  Hits         1940     1940           
  Misses        428      428           
  Partials      232      232           
Impacted Files Coverage Δ
xbout/load.py 78.15% <ø> (ø)
xbout/boutdataarray.py 76.85% <100.00%> (ø)

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 8f14cb2...becd461. Read the comment docs.