search

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

animate_list() method to animate multiple plots at once. #73

Closed johnomotani closed 4 years ago

johnomotani commented 4 years ago

Split out from #38. Should be merged after #72.

Replaces #38.

pep8speaks commented 4 years ago

Hello @johnomotani! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-12-10 23:32:43 UTC
codecov-io commented 4 years ago

Codecov Report

Merging #73 into master will decrease coverage by 0.66%. The diff coverage is 35.8%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   45.15%   44.49%   -0.67%     
==========================================
  Files           8        8              
  Lines         795      971     +176     
  Branches      152      200      +48     
==========================================
+ Hits          359      432      +73     
- Misses        380      481     +101     
- Partials       56       58       +2
Impacted Files Coverage Δ
xbout/plotting/utils.py 6.91% <11.53%> (+1.99%) :arrow_up:
xbout/plotting/animate.py 43.93% <24.61%> (-10.94%) :arrow_down:
xbout/plotting/plotfuncs.py 7.47% <5.71%> (-1.86%) :arrow_down:
xbout/boutdataarray.py 62.96% <52.94%> (-8.47%) :arrow_down:
xbout/boutdataset.py 63.7% <81.66%> (+16.83%) :arrow_up:

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 010274a...308c568. Read the comment docs.