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

dtc/develop: combined PRs for ccpp-framework and ccpp-physics #21

Merged
merged 2 commits into from
Feb 6, 2020

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Feb 5, 2020

This PR updates the submodule pointer for fv3atm and .gitmodules (for code review and testing, will be reverted after the submodules are merged).

Associated PRs:
NCAR/ccpp-framework#260
NCAR/ccpp-physics#394
NCAR/fv3atm#25
#21

See below for regression testing information.

@climbfuji
Copy link
Collaborator Author

Regression testing on hera.intel using full rt.conf (IPD PROD, IPD REPRO, CCPP REPRO, CCPP PROD) against DTC baseline (which is the same as the EMC baseline, but with the IPD REPRO and CCPP PROD results): all pass

rt_full.log

@climbfuji
Copy link
Collaborator Author

Regression testing on hera.intel using rt_ccpp_dtc.conf (first create, then verify): all pass

rt_ccpp_dtc_create.log
rt_ccpp_dtc_verify.log

@climbfuji
Copy link
Collaborator Author

Regression testing on hera.intel using rt_ccpp_hafs.conf (first create, then verify): all pass

rt_ccpp_hafs_verify.log
rt_ccpp_hafs_create.log

@climbfuji
Copy link
Collaborator Author

Regression testing on hera.intel using rt_ccpp_caps.conf (first create, then verify): all pass

rt_ccpp_caps_verify.log
rt_ccpp_caps_create.log

@climbfuji climbfuji merged commit 2919f28 into NCAR:dtc/develop Feb 6, 2020
SamuelTrahanNOAA pushed a commit to SamuelTrahanNOAA/ufs-weather-model that referenced this pull request May 14, 2020
Number concentration bug and code clean up in ccpp-physics GFS_suite_interstitial_4
climbfuji pushed a commit that referenced this pull request Jun 8, 2020
* Remove unused and unsupported code (See #21)
* Remove APPBUILD step from rt.conf
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.

2 participants