You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Just realized that the error message can be more specific to add the freq for the second check, so maybe we include that in this next PR also:
# Check if the specified freq-pulse length combination is in the allowable Sv_offset dict
if pulse_len not in SV_OFFSET[freq]:
raise ValueError(
f"Pulse length {pulse_len} us is not in the Sv offset dictionary "
"for the {freq} Hz channel! " # add freq info
"Please contact AZFP Environmental Sciences "
"and raise an issue in the echopype repository."
)
Hey @ctuguinay - I just noticed this. I think we have the test files now from an email from ASL. Could you add that to the test suite and we can close this? Thanks!
Sv_offset
dataset in the AZFP parser and pack it inparser_obj.unpacked_data
_calc_Sv_offset
for AZFP parsing #1304The text was updated successfully, but these errors were encountered: