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

anmn_temp_gridded_product.py - Static max depth per site #698

Open
ggalibert opened this issue Jun 1, 2017 · 7 comments
Open

anmn_temp_gridded_product.py - Static max depth per site #698

ggalibert opened this issue Jun 1, 2017 · 7 comments

Comments

@ggalibert
Copy link
Contributor

ggalibert commented Jun 1, 2017

It would be good to set the maximum depth of the T re-gridded product consistent across different deployments. Say for example site nominal depth + 10%. So for a site which nominal depth is 100m, the max depth for which the product can give data would be 110m.

It is important to hard code site nominal depth somewhere and not rely on the value given in the global attributes since this can be erroneously/inconsistently documented.

@lbesnard
Copy link
Contributor

lbesnard commented Jan 8, 2018

How would you see this being hard coded ? should we have a text file with this information ?

site_code = nominal_depth ?

@ggalibert
Copy link
Contributor Author

How would you see this being hard coded ? should we have a text file with this information ?
site_code = nominal_depth ?

Yes, chosen nominal_depth could be the deepest found so far in all the deployments of a site.

@lbesnard
Copy link
Contributor

lbesnard commented May 2, 2019

I guess we can close this now @ggalibert

@ggalibert
Copy link
Contributor Author

Not yet!

@ocehugo
Copy link
Contributor

ocehugo commented May 6, 2019

2 cents:

I don't see a reason why the 10% rule. The nominal depth of the site will never be reached by any measurement - even bottom ADCP are some meter above the floor.
If any measurement falls into or below the nominal depth this helps indicate an error on the depth value of either the deployment or the instrument.

I think it would be a good standard to set the x(z) bottom most value to masked at z == nominal depth.

@ggalibert
Copy link
Contributor Author

Some sites can be located on steep continental slope so sampled depth and site depth can vary across deployments.

One feedback was that users don't want to retrieve products that have inconsistent vertical spans if requested over different time periods, hence the idea of hard coding the nominal site depth + 10% or so in order to set a consistent vertical span per site.

@lbesnard lbesnard removed their assignment Jun 11, 2019
@ocehugo
Copy link
Contributor

ocehugo commented Nov 3, 2020

Shall we close!? Code is now deprecated and the task can be done by the end user by combining/subsetting the long-time series products.
@ggalibert , @lbesnard

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

No branches or pull requests

3 participants