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

Better values for jyseps2_2 in test data #185

Closed johnomotani closed 3 years ago

johnomotani commented 3 years ago

In core, sol or limiter configurations, BOUT++ would set jyseps2_2=ny-1, rather than ny which was previously used in test_load.py. The difference should not be important for anything at the moment, but might possibly hide a bug in future if not updated.

codecov-io commented 3 years ago

Codecov Report

Merging #185 (bcfca4d) into master (3748171) will decrease coverage by 0.08%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   75.89%   75.81%   -0.09%     
==========================================
  Files          14       14              
  Lines        2394     2394              
  Branches      553      553              
==========================================
- Hits         1817     1815       -2     
- Misses        380      381       +1     
- Partials      197      198       +1     
Impacted Files Coverage Δ
xbout/region.py 84.66% <0.00%> (-0.44%) :arrow_down:

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 3748171...bcfca4d. Read the comment docs.