Edits agreed by the CDM_OBS Governance group - 24th April 2025 #4
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
edits units table entry 301 to be ppt rather than 0/000 agreed by CDM OBS governance April 2025
Added entries 20 and 21 to observations_fields table as agreed by the CDM_OBS governance in April 2025
eidts agreed by CDM_OBS Governance board May 2025
Edits agreed by the CDM-OBS Governance board May 2025
Edits agreed by the CDM-OBS Governance board May 2025
Edits agreed by the CDM-OBS Governance board May 2025
|
|
EddyCMWF
reviewed
Jun 2, 2025
| @@ -46,7 +46,9 @@ units name abbreviation base_units | |||
| 230 week NULL | |||
| 231 year a NULL | |||
| 300 per cent \% NULL | |||
| 301 parts per thousand 0/00 NULL | |||
| 301 parts per thousand ppt NULL | |||
There was a problem hiding this comment.
My experience is that ppt = parts per trillion. This is backed up by wikipedia. However, this is more a point for the governance board. permille (‰) is the more generally accepted units
There was a problem hiding this comment.
Good point! and the symbol exists... we can make that correction.
EddyCMWF
reviewed
Jun 5, 2025
| @@ -46,7 +46,9 @@ units name abbreviation base_units | |||
| 230 week NULL | |||
| 231 year a NULL | |||
| 300 per cent \% NULL | |||
| 301 parts per thousand 0/00 NULL | |||
| 301 parts per thousand ppt NULL | |||
There was a problem hiding this comment.
Suggested change
| 301 parts per thousand ppt NULL | |
| 301 parts per thousand \‰ NULL |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
Sorry for the delay.
Here are the edits agreed at the CDM_OBS Governance group call on the 24th April 2025.
Here is the link to boards page with all notes and agreed actions for reference.
https://board.net/p/C3S_311_CDM-OBS_Governance_call_24th_April
Can you please review the edits and merge them into the master if everything looks good?
Let me know if you have any questions or need clarification!
best
Simon