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

add pressure levels to long name #42

Open
larsbuntemeyer opened this issue Oct 7, 2024 · 0 comments
Open

add pressure levels to long name #42

larsbuntemeyer opened this issue Oct 7, 2024 · 0 comments
Assignees

Comments

@larsbuntemeyer
Copy link
Contributor

larsbuntemeyer commented Oct 7, 2024

Standard names must follow those defined by CF, and these represent the physical quantity. Other aspects, such as the level, are represented by other metadata of the variable (such as a scalar coordinate variable representing the pressure level). So standard_names are OK even if repeated for many different variables.

Long names are not critical as these are just free text descriptions. However, you spotted a clear inconsistency between the variable and the corresponding metadata, so this should be fixed. I let the PR open, just in case you found some others.

Long names without much detail are no problem, in principle. For pressure level data we could include the level explicitly, though. E.g. in CMIP6 they do include it:
https://github.com/PCMDI/cmip6-cmor-tables/blob/80d2e4b606555720630a61389cfe3b5cb8f43746/Tables/CMIP6_6hrPlev.json#L312

@larsbuntemeyer @gnikulin what do you think?

Originally posted by @jesusff in #39 (comment)

@larsbuntemeyer larsbuntemeyer self-assigned this Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant