Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

issues with Doppler current profiler descriptions #189

Closed
jklymak opened this issue Feb 1, 2021 · 4 comments
Closed

issues with Doppler current profiler descriptions #189

jklymak opened this issue Feb 1, 2021 · 4 comments
Assignees
Labels
standard name (added by template) Requests and discussions for standard names and other controlled vocabulary

Comments

@jklymak
Copy link

jklymak commented Feb 1, 2021

In #72 you very kindly added CF names for acoustic Doppler current profilers, which is super helpful. This data needs to appear in ADCP files to affect meaningful quality control.

There are some misleading statements in the descriptions that could perhaps be modified. Reference to https://www.comm-tec.com/Docs/Manuali/RDI/BBPRIME.pdf

beam_consistency_indicator_from_multibeam_acoustic_doppler_velocity_profiler_in_sea_water

The "beam_consistency_indicator" is the degree to which the magnitudes of a collection (ensemble) of acoustic signals from multiple underwater acoustic transceivers relate to each other.

To me this makes it sound like the pings are compared to each other, but in fact they are being individually correlated with the transmitted pulse. The strength of the correlation is a signal-to-noise quality, and the phase of the correlation is what is used to measure the Doppler shift:

The "beam_consistency_indicator" is the degree to which the received acoustic pulse is correlated with 
the transmitted pulse.  

I guess I think this was unfortunately named, but I don't know what your scope is to change this now...

indicative_error_from_multibeam_acoustic_doppler_velocity_profiler_in_sea_water

 It is determined by differencing duplicate error velocity measurements made using different pairs of beams.

This isn't correct either. You can form a solution for u,v, w, with three beams, and the ADCPs usually have 4, so the error velocity is the absolute difference in w between two 3-beam solutions. Perhaps:

It is determined by the difference between the vertical velocity calculated from two 3-beam solutions.

Thanks a lot!

@jklymak jklymak added the standard name (added by template) Requests and discussions for standard names and other controlled vocabulary label Feb 1, 2021
@roy-lowry
Copy link

Many thanks for the clarifications. I support their replacing the existing text in the descriptions.

@feggleton
Copy link
Collaborator

I have added these changes in the cfeditor (http://cfeditor.ceda.ac.uk/proposals/1). Please comment if there is any further discussion. If there are no further comments in the next 7 days I think these small changes can be accepted in order to clarify the definitions.

@feggleton
Copy link
Collaborator

These two terms have now been accepted.

@japamment
Copy link
Member

Changes applied in version 78 of the standard name table.

@efisher008 efisher008 transferred this issue from cf-convention/discuss Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
standard name (added by template) Requests and discussions for standard names and other controlled vocabulary
Projects
None yet
Development

No branches or pull requests

4 participants