Closed johnomotani closed 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:
Merging #134 into master will increase coverage by
0.19%
. The diff coverage is79.42%
.
@@ Coverage Diff @@
## master #134 +/- ##
==========================================
+ Coverage 77.44% 77.64% +0.19%
==========================================
Files 14 14
Lines 1827 2035 +208
Branches 393 452 +59
==========================================
+ Hits 1415 1580 +165
- Misses 277 299 +22
- Partials 135 156 +21
Impacted Files | Coverage Δ | |
---|---|---|
xbout/boutdataset.py | 74.40% <65.38%> (-0.88%) |
:arrow_down: |
xbout/utils.py | 81.56% <77.51%> (-10.44%) |
:arrow_down: |
xbout/boutdataarray.py | 85.92% <90.74%> (+1.13%) |
: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 e229046...533ab4b. Read the comment docs.
Adds a method to interpolate data from the BOUT++ grid onto a different structured or unstructured grid. The structured grid can be useful to get Cartesian output, e.g. for cross-code comparison or 3d visualisation. Unstructured output can be useful e.g. for interpolating to the measurement positions of some diagnostic.
Should be merged after #130.