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

add capability to specify pressure sensor height for a vector #181

Closed
rallen-usgs opened this issue Jan 17, 2024 · 7 comments
Closed

add capability to specify pressure sensor height for a vector #181

rallen-usgs opened this issue Jan 17, 2024 · 7 comments

Comments

@rallen-usgs
Copy link
Contributor

Vectors can be oriented horizontally or vertically (up or down), and they can have internal or external pressure sensors. I think it may be useful to specify the pressure sensor height above bed along side the vector head. Perhaps we can add a key to the yaml file along the lines of "pressure_sensor_height"?

@dnowacki-usgs
Copy link
Member

CC @ssuttles-usgs thoughts on how to implement this?

@ssuttles-usgs
Copy link
Collaborator

Looking back to processing Marinna did for our dataset from Matanzas Inlet in 2018, she used these 2 attributes in the old matlab SVN code to specify the probe and pressure sensor height for cabled vector deployments.

metadata.pressure_sensor_height = 2.008; % m
metadata.probe_height = 0.553; % m

I suggest we implement something similar in stglib. We routinely deploy cabled vectors where the heights of the corresponding pressure data needs to be specified in addition to the velocity probe (or measurement volume) height. In fact, we have a couple of datasets in the processing queue with cabled vectors.

@dnowacki-usgs
Copy link
Member

I'm checking in with PCMSC folks to see if they used the same nomenclature in their Matlab scripts. I wonder if probe_height is the preferred way to specify the height. Should it be sample volumes height?

Would we then have a z coordinate variable with two values, one for pressure and one for probe sample volume, and then index the appropriate variables by time, z?

@ssuttles-usgs
Copy link
Collaborator

I think you would need a second height coordinate variable, right? maybe 'zp'? or alternately have a vertical offset attribute in the pressure variable? Not sure what is better.

@dnowacki-usgs
Copy link
Member

PCMSC uses the same nomenclature in the Matlab scripts.

metadata.pressure_sensor_height = 0.62;
metadata.probe_height = 0.40; %vec cmab on frame %not sv

Note that specifies height of probe and not SV. What's the best approach here? [probe or SV?]

@dnowacki-usgs
Copy link
Member

I think it's better to specify the pressure sensor height and the sample volume height [not probe height] for velocity. So I propose pressure_sensor_height and velocity_sample_volume_height as the attrs to specify these.

@dnowacki-usgs
Copy link
Member

This has been included in #186, so closing this issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants