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

adcpBinMappingPP for new Nortek Signature #651

Open
sspagnol opened this issue Mar 11, 2020 · 4 comments
Open

adcpBinMappingPP for new Nortek Signature #651

sspagnol opened this issue Mar 11, 2020 · 4 comments
Labels
Type:bug Something is missing or wrong Unit:Instrument Reader Instrument parsers and related Unit:Processing preprocessing and transformations
Milestone

Comments

@sspagnol
Copy link
Contributor

If I understand adcpBinMappingPP correctly if will bin map 4 beam RDI and 3 beam Nortek, as in convert DISTANCE_ALONG_BEAMS to HEIGHT_ABOVE_SENSOR, but I don't think it does this for new 4 beam Nortek Signatures.

@ocehugo ocehugo added Unit:Instrument Reader Instrument parsers and related Unit:Processing preprocessing and transformations labels Mar 12, 2020
@ocehugo
Copy link
Contributor

ocehugo commented Mar 12, 2020

@sspagnol , so this is a bug (we are remapping wrongly) or a feature (we can't map atm but will need to remap soonish)?

Also, could be a bit more specific about the instrument model & stacktrace of the toolbox (if any) + example file?

@sspagnol
Copy link
Contributor Author

Attached is Nortek Signature 500 ad2cp file.

S100807A004_ROW01_avgd.zip

It was part of new NWSROW (Rowley Shoals) mooring (oriented down), which included an RDI (oriented up). When processed the RDI nc file had HEIGHT_ABOVE_SENSOR dimension but the Nortek didn't. So trying to understand did I miss something or is adcpBinMappingPP not working as expected.

@ocehugo ocehugo added the Type:bug Something is missing or wrong label Mar 12, 2020
@ocehugo
Copy link
Contributor

ocehugo commented Mar 12, 2020

Thanks.

I just had a quick look and, yes, we are totally skipping when we found ABSIC4 in Norteks. Logic is incomplete for signatures - there is no code that handles 4 beam cases.

Probably something simple, but I need some time to dig into adcpBinMappingPP and the specs/mapping trigonometry to do it right.

Requirements:

  • review Instrument specs and trigonometry/beam angle configuration.
  • Implement logic for Nortek Signatures in adcpBinMappingPP.

@ocehugo
Copy link
Contributor

ocehugo commented Mar 12, 2020

I knew this was somewhere - see #531

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:bug Something is missing or wrong Unit:Instrument Reader Instrument parsers and related Unit:Processing preprocessing and transformations
Projects
None yet
Development

No branches or pull requests

2 participants