-
Notifications
You must be signed in to change notification settings - Fork 5
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
Comments
How would you see this being hard coded ? should we have a text file with this information ?
|
Yes, chosen |
I guess we can close this now @ggalibert |
Not yet! |
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. I think it would be a good standard to set the x(z) bottom most value to masked at z == nominal depth. |
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. |
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. |
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.
The text was updated successfully, but these errors were encountered: