Closed Mikejmnez closed 1 year ago
Merging #300 (0a76507) into main (5663338) will increase coverage by
0.49%
. The diff coverage is92.18%
.
@@ Coverage Diff @@
## main #300 +/- ##
==========================================
+ Coverage 95.25% 95.74% +0.49%
==========================================
Files 9 9
Lines 3898 3928 +30
Branches 833 850 +17
==========================================
+ Hits 3713 3761 +48
+ Misses 116 97 -19
- Partials 69 70 +1
Flag | Coverage Δ | |
---|---|---|
unittests | 95.74% <92.18%> (+0.49%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
oceanspy/compute.py | 98.81% <ø> (+1.07%) |
:arrow_up: |
oceanspy/subsample.py | 98.04% <ø> (ø) |
|
oceanspy/llc_rearrange.py | 88.60% <92.00%> (+0.19%) |
:arrow_up: |
oceanspy/utils.py | 86.86% <92.85%> (+4.43%) |
:arrow_up: |
oceanspy/_oceandataset.py | 97.12% <0.00%> (-0.24%) |
:arrow_down: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR addresses issues #285 , #291 and #299. In particular:
llc_rearrange.LLC_Transformation.arctic_crown
). As a result, the new od.grid of the resulting transformed dataset always hasinner
andouter
as grid_coords.find_cs_sn
fromospy.utils
.local_to_latlon
is replaced byllc_local_to_lat_lon
withinllc_rearrange
. This new function can rotate all vector fields and has a grid consistent with a staggered c-grid.viewer_to_range
can now take all arguments from poseidon viewer. Included tests for this.