Closed emiliom closed 10 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
71f0250
) 83.44% compared to head (14155b1
) 77.96%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
And yeah,
absorption_indicative
andsound_speed_indicative
need to be calculated for AZFP for calibrating the data.
Ah. Maybe in the next release, the calculation of those parameters can be moved to open_raw
, if appropriate.
I'll merge the PR now. Thanks!
Fully addresses #1225.
I'm not aware of a source of data to populate the
absorption_indicative
&sound_speed_indicative
variables, so I'm simply filling them withnp.nan
. @leewujung do you know if there's something in the AZFP raw data & XML that could be used? If there isn't, or it's not straightforward, I suggest leaving the variables in withnp.nan
; after all, the variables were not present before this PR :sweat_smile: