-
Notifications
You must be signed in to change notification settings - Fork 1
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
dealing with ranges in the logsheets #21
Comments
'<' --> max depth change ontology and templates to include this (not for the depth in observatory logsheets/data/templates - rather sampling tab) |
@kmexter will check whether there are any max/min occurring in measurments |
For the QC there is a check that depth is < dpeth max, for this the check will either not check it (treat as a string float - float) or will check the max depth against the value from observatory tab. |
I think it's more appropriate if we keep the maximum depth for the ENA submission. |
Discussed with @bulricht: this is now to be treated as a string, however as there is a QC check for this against the tot_depth_water_col in the observatory tab of each logsheet, this will have to be done in a new way: |
@laurianvm if you have finished your part of this issue, can you say so here so I know that it is only Bram's part left to do? |
Hmm. @laurianvm :-} we think it is better that rather than you transforming these depths into a max and min, it is better that it is done to the transformed loghsheets - we create 2 new columns, depth_min and depth_max. That may mean you backpeddling on some template changes? We think that because it is best to do the least amount of data changes in the templates as possible. |
@kmexter to find new BODC terms for these for logsheets transformed |
@kmexter raise new issue to say how to do this when some entries are a range and some are single values I should double check if there are any unbounded ranges, and if so needs to be a range instead (0-5 = < 5 and 5-maxdepth - >5) And tell the stations tell them how to indicate a range: 3-5. |
We have discussed this in other issues but moving here to reduce the confusion
There are some values in the logsheets that are entered as a <, >, or - (range)
For the size fractions (size_frac_up and low) this is just wrong and should be flagged as such in the QC. There is an upper and a lower value, and if one of them is not known (as it did not exist, so to speak), then the value in that cell should be either NA or blank.
--> @kmexter make sure that this is not only flagged in the QC, but also transformed into just a value AND that any NA or 0 values are given a "no value for this" in the triples. @laurianvm may be able to advise on this.
For depth, we will take the ranges given and translate them into an upper and lower limit, and if we do get a < or > we will do the same, but only inthe TTL files - in the transformed logsheets these will continue to be just the string 3-4 or <10 or whatever. To be consistent over all stations, we will make all depths a max and a min. @laurianvm can you take care of this in your template - if not, let's chat
For the enviro measurements: at the next harvest I will ask that this is flagged specifically and so I will see if there are any like this (was it my imagination that I saw this?). If there are, we will deal with that in the ttl file only, by making that a "max" value
The text was updated successfully, but these errors were encountered: