OSOceanAcoustics / echopype

Enabling interoperability and scalability in ocean sonar data analysis
https://echopype.readthedocs.io/
Apache License 2.0
99 stars 73 forks source link

Rename `fs_receiver` in Vendor-specific group #1104

Closed leewujung closed 1 year ago

leewujung commented 1 year ago

This PR rename the data variable fs_receiver in the Vendor-specific group to receiver_sampling_frequency. This should have been changed around the time of #1019, but somehow didn't get fixed.

codecov-commenter commented 1 year ago

Codecov Report

Merging #1104 (0280f8d) into dev (13be688) will decrease coverage by 12.11%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              dev    #1104       +/-   ##
===========================================
- Coverage   78.12%   66.01%   -12.11%     
===========================================
  Files          65       30       -35     
  Lines        6227     3911     -2316     
===========================================
- Hits         4865     2582     -2283     
+ Misses       1362     1329       -33     
Flag Coverage Δ
unittests 66.01% <100.00%> (-12.11%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
echopype/calibrate/cal_params.py 91.79% <100.00%> (ø)
echopype/convert/set_groups_ek80.py 82.52% <100.00%> (-14.50%) :arrow_down:

... and 53 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more