-
Notifications
You must be signed in to change notification settings - Fork 20
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
Register species from MICM #102
Register species from MICM #102
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.
Minor comment -- We are trying to standardize the ordering of the error message and error code in CCPP (just for uniformity sake). The ordering that was decided upon is errmsg, errcode, mainly because the NOAA code base uses this ordering. See NCAR/ccpp-framework#574
done! |
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!
Uses updated CCPP-generated data to include scheme dependencies in the CAM-SIMA build. In draft until ESCOMP/atmospheric_physics#102 is merged. --------- Co-authored-by: Jesse Nusbaumer <[email protected]>
Registers constituents and sets needed constituent properties from MICM configuration.
(reopened to merge to development branch)
Additionally:
returns error codes and messages from MUSICA functions using new error handling in MICM
creates a musica_ccpp module to act as a the CCPP interface and moves existing functionality to a musica_ccpp_micm module in preparation for adding TUV-x (we will use musica_ccpp_ as a prefix for all the MUSICA modules in this repo to prevent naming conflicts with MUSICA Fortran library modules)
updates build scripts for changes to MUSICA and CCPP framework
adds memory checking to MUSICA tests
closes #74