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

Bugfix for `_pad_x_boundaries()` #184

Closed johnomotani closed 3 years ago

johnomotani commented 3 years ago

Was missing () on call to ds.load().

codecov-io commented 3 years ago

Codecov Report

Merging #184 (b3d34f6) into master (4f84abb) will increase coverage by 0.92%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
+ Coverage   74.97%   75.89%   +0.92%     
==========================================
  Files          12       14       +2     
  Lines        2066     2394     +328     
  Branches      476      553      +77     
==========================================
+ Hits         1549     1817     +268     
- Misses        347      380      +33     
- Partials      170      197      +27     
Impacted Files Coverage Δ
xbout/utils.py 81.73% <ø> (ø)
conftest.py
xbout/__init__.py 81.25% <0.00%> (ø)
xbout/conftest.py 76.92% <0.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 4f84abb...b3d34f6. Read the comment docs.

johnomotani commented 3 years ago

Thanks for the review @Vandoo!