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

CCPP API and SDF update #56

Merged
merged 7 commits into from
Apr 25, 2018
Merged

CCPP API and SDF update #56

merged 7 commits into from
Apr 25, 2018

Conversation

climbfuji
Copy link
Collaborator

This PR contains several changes to the CCPP API and SDF (suite definition file) for all host models.

This PR goes together with PRs for ccpp-physics, gmtb-scm and gmtb-fv3 listed below.

For a detailed description see NCAR/ccpp-physics#74

…oup' (previously called 'ipd'), use strings describing the group. This requires new functionality in ccpp_fcall.F90 to find a group or scheme by its name. The single entry point for running physics is now ccpp_physics_run with mandatory arguments cdata and ierr, and optional arguments describing what to run (a group, a subcycle of a group, a single scheme, or the entire suite). Corresponding updates for the suite definition files of SCM and FV3.
@climbfuji
Copy link
Collaborator Author

Copy link
Contributor

@llpcarson llpcarson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@climbfuji
Copy link
Collaborator Author

The Travis CI build fails because of an intentional failure of one of the tests (suite_noop_3.xml). This will be fixed in a follow-up PR (either remove the test or assert failure).

@climbfuji climbfuji merged commit 66f1a06 into NCAR:master Apr 25, 2018
@climbfuji climbfuji deleted the ccpp-api-update branch April 25, 2018 14:54
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

Successfully merging this pull request may close these issues.

3 participants