nvs-vocabs / ArgoVocabs

A repository for the management of issues related to vocabularies managed by the Argo Data Management Team
7 stars 0 forks source link

R03: Should units be part of the description and a preferred unit string ? #74

Open BrianKingNOC opened 8 months ago

BrianKingNOC commented 8 months ago

Here is another R03 issue. This is prompted by the json metadata activity, but not confined to it.

Table R03 doesn't refer to the units for a parameter. The xls table that proceeded it had a lot of extra information about range (min/max) format/resolution etc. Range and numerical format have already had issues raised.

Now it is the turn for units.

For example the xls spreadsheet tells me TEMP should be degree_Celsius although I need to go to another column to also see that that is in ITS-90.

This units info isn't actually recorded anywhere in the meta.nc. @jean-philippe Ranou pointed out to me that PARAMETER_UNITS is supposed to refer to PARAMETER_ACCURACY And PARAMETER_RESOLUTION but not necessarily PARAMETER.

So the whole issue of where we record units is a bit incomplete. As far as I can tell, none of the Rxx tables, useable by machines, tell a machine what the units should be in Argo files, unless I have missed something.

In the data files, eg prof.nc, the PARAMETER has attributes including units, and the User Manual says those fields are specified in ref table 3. But they're not in NVS R03.

tcarval commented 8 months ago

For each R03 parameter, the unit is available as "related P06: *" Example : https://vocab.nerc.ac.uk/collection/R03/current/TEMP/

Related | P06:[UPAA ](https://vocab.nerc.ac.uk/collection/P06/current/UPAA/)| Degrees Celsius -- | -- | --

Do we miss something there ?

BrianKingNOC commented 8 months ago

Aha. Thanks @tcarval

Looking at the page for TEMP http://vocab.nerc.ac.uk/collection/R03/current/TEMP/ I didn't realise that related P06 would take me to units. That comes with experience I guess.

As it happens, TEMP_CNDC doesn't have the 'related' link. http://vocab.nerc.ac.uk/collection/R03/current/TEMP_CNDC/

So if I follow TEMP, I find units are "Degrees Celsius" alt label "degC"

and for CNDC, I find "Siemens per metre" with alt "S/m"

I saw the old xls sheet Argo physical parameters list: core-Argo and BGC-Argo, February 3rd 2023 has "degree_Celsius" for all the temperatures parameters and "mhos/m" for CNDC.

I'm quite happy to let P06 become the master reference. I notice the user manual Page 23 has "degree_Celsius" in the example.

I guess the units attribute in the prof.nc file is autofilled for all the parameters. Is there a plan to ask DACs to move over to the approved names in P06 linked from the parameters in R03 ?

In the json work, we might introduce a new unit field to accompany the parameter, chosen from R03/P06, to at least get things correct from the origin of the sensor.

tcarval commented 8 months ago

There is no plan to change the Argo units labels. We'll check what is the best option on NVS to list the existing Argo units. @vpaba , @gwemon, ideas ?