Open lsetiawan opened 6 months ago
Attention: Patch coverage is 95.29412%
with 4 lines
in your changes are missing coverage. Please review.
Project coverage is 70.38%. Comparing base (
8600a59
) to head (bb257cc
). Report is 38 commits behind head on dev.
Files | Patch % | Lines |
---|---|---|
echopype/commongrid/regrid.py | 97.43% | 2 Missing :warning: |
echopype/utils/misc.py | 33.33% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@leewujung This is currently a draft PR because I don't have tests for it yet. However, could you please start reviewing the implementation and help me try out different scenarios to make sure I have it implemented correctly? Thanks!
@lsetiawan : Could you put in at least a couple test so that it's easier to step through the code? Thanks!
Overview
This PR add regridding functionality for existing Sv data