-
Notifications
You must be signed in to change notification settings - Fork 132
Cmorizer for HALOE #1581
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
Cmorizer for HALOE #1581
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. How difficult would it be to add the other variables? Would that just be an extension of the code?
Just extend the arrays |
Thanks!! You are the best! |
I think a new release of ESMValCore is required for the tests to pass, due to the new custom table introduced in ESMValGroup/ESMValCore#567. |
if (VAR(vv).eq."hus") then | ||
xx = xx * 18.015 / 28.97 ; [mol/mol] --> [kg/kg] | ||
xx = xx / (1. + xx) | ||
end if |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@axel-lauer @hb326
Can you please cross-check the unit conversion?
See here for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks correct. E.g. for CMIP6, it is explicitely in the CMOR tables that hus is given in kg (H2O) per kg (moist air):
"comment": "Specific humidity is the mass fraction of water vapor in (moist) air."
so the conversion of the HALOE data from m3/m3 to kg(H2O)/kg(moist air) as done here is correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Please approve so I can merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks all fine. Double-checked conversion of variable ‚hus’, looks correct.
…lTool into cmorizer_haloe
Before you start, read CONTRIBUTING.md and the guide for diagnostic developers.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.
Tasks
yamllint
to check that your YAML files do not contain mistakesNew recipe/diagnostic
doc/sphinx/source/recipes
folder and add a new entry to index.rstModified recipe/diagnostic
doc/sphinx/source/recipes
folderNew data reformatting script
Modified data reformatting script
If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.
Requires ESMValGroup/ESMValCore#567