Closed diodon closed 4 years ago
@diodon , comments up for grabs!
:exclamation: No coverage uploaded for pull request base (
master@14b0f24
). Click here to learn what that means. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #90 +/- ##
=========================================
Coverage ? 82.46%
=========================================
Files ? 6
Lines ? 998
Branches ? 192
=========================================
Hits ? 823
Misses ? 135
Partials ? 40
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 14b0f24...5f47f6e. Read the comment docs.
Wouldn't you dump the groupby call? Nonetheless, I can't merge since you need to fix the conflicts.
grid TEMP or PSAL values from hourly timeseries product into fixed target depths.