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

CTD_TEMP_CNDC in R25 #50

Closed
apswong opened this issue Dec 13, 2022 · 5 comments
Closed

CTD_TEMP_CNDC in R25 #50

apswong opened this issue Dec 13, 2022 · 5 comments
Assignees
Labels
avtt Argo Vocabulary Task Team

Comments

@apswong
Copy link

apswong commented Dec 13, 2022

I would like to request a new entry in R25 "SENSOR". It is CTD_TEMP_CNDC.

TEMP_CNDC is the internal temperature of the conductivity cell. For RBR CTDs, measurements for TEMP and TEMP_CNDC come from two separate thermistors. DACs will need CTD_TEMP_CNDC to record predeployment calibration information in the meta file separately from CTD_TEMP.

@vpaba vpaba added the avtt Argo Vocabulary Task Team label Jan 26, 2023
@vpaba vpaba self-assigned this Feb 9, 2023
@BrianKingNOC
Copy link

BrianKingNOC commented Oct 9, 2023

@vpaba @tcarval I'd like to reactivate this

Our work with sensor makers to distribute complete metadata that is consistent with NVS is maturing, and looks likely to involve live checking of some fields against NVS tables. Therefore the tables need to be complete, and CTD_TEMP_CNDC is needed in the online version of R25.

@tcarval
Copy link
Contributor

tcarval commented Oct 22, 2023

@apswong , @BrianKingNOC : All temperature parameter names start with "TEMP"
Can we add "TEMP_CNDC_CTD" : "External temperature of the conductivity sensor"
Can you confirm that on a RBR sensor, there is a temperature sensor on the CTD and another distinct temperature sensor used with CNDC ?

@BrianKingNOC
Copy link

BrianKingNOC commented Oct 22, 2023

@tcarval : R25 is the list of SENSORs not PARAMETERs

SENSORs tend to begin with something to do with the instrument, eg CTD_TEMP or STS_TEMP or TRANSMISSOMETER_CP660

So I would suggest the SENSOR should definitely be CTD_TEMP_CNDC, and yes it is a different device that measures temperature in a different place, and will likely have its own SENSOR_SERIAL_NO

The PARAMETER it measures can be TEMP_CNDC, which sits in R03 alongside TEMP_DOXY and TEMP_CPU_CHLA, starting with TEMP

@tcarval
Copy link
Contributor

tcarval commented Oct 22, 2023

My mistake, I'll add CTD_TEMP_CNDC in R25

tcarval added a commit to euroargodev/ArgoNetCDF that referenced this issue Oct 23, 2023
@tcarval
Copy link
Contributor

tcarval commented Oct 23, 2023

I added CTD_TEMP_CNDC in R25 (both NVS and format checker)

  • Identifier: CTD_TEMP_CNDC
  • Prefered label: Temperature sensor for conductivity on a CTD
  • Definition: Specific temperature sensor used with conductivity to calculate salinity on a CTD

New release of the format checker:
https://github.com/euroargodev/ArgoNetCDF/releases/tag/v2.8.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
avtt Argo Vocabulary Task Team
Projects
Development

No branches or pull requests

4 participants