-
Notifications
You must be signed in to change notification settings - Fork 14
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
Includes CCPP scheme dependencies in the build #282
Includes CCPP scheme dependencies in the build #282
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, but did have one change request, which I am happy to discuss if need be. Thanks!
@mattldawson Are you ok with me pushing to this PR? I found one small issue when running our regression tests. |
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 @mattldawson !
small change and submodule question
Uses updated CCPP-generated data to include scheme dependencies in the CAM-SIMA build.
In draft until ESCOMP/atmospheric_physics#102 is merged.