-
Notifications
You must be signed in to change notification settings - Fork 12
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
Integration of CCPP-ized TJ2016 implementation #271
Conversation
…ing gravit standard name. Making etamid visible via metadata file.
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! I know it appears like a decent number of change requests but most of them should just be related to changing a particular error message.
Also, can the PR description be updated to include the following two things:
-
The PR URL (so we can grab it in the git log if need be, unless Github will do this for us?)
-
The results of the regression tests run on Derecho.
Instructions for how to run the tests can be found here:
https://github.com/ESCOMP/CAM-SIMA/wiki/CAM-SIMA-testing#regression-testing-manual
Note that you'll need to do it twice, once for GNU and once for Intel. It might also be good to do snapshot test runs with the "official" snapshots that will live on Derecho here:
/glade/campaign/cesm/community/amwg/sima_baselines/cam_sima_test_snapshots
Mostly we just want to have a record that everything passed.
Thanks, and please let me know if you have any questions or concerns with any of my suggestions!
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 great, thanks @mwaxmonsky!
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.
two more tiny error checks!
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!
@peverwhee @nusbaume I fixed the merge conflicts and brought in the git-fleximod updates but if either of you had a sec, could you double check I didn't break anything with the fleximod update? Everything works on my end but I just want to make sure the git update works as expected on the user side. |
@mwaxmonsky I ran a clean check-out of CAM-SIMA with your updates and git-fleximod worked well. I did run into this bug - #280 (introduced by me), but I vote for merging this PR! |
Updates atmospheric_physics to the latest tag bringing in TJ2016 capability and additional changes needed in support of TJ2016.
Describe any changes made to build system: N/A
Describe any changes made to the namelist: N/A
List any changes to the defaults for the boundary datasets: N/A
Describe any substantial timing or memory changes: N/A
Code reviewed by: nusbaume, peverwhee
List all existing files that have been modified, old files eliminated, new files added and describe the changes:
M Externals_CAM.cfg
M src/control/cam_comp.F90
M src/data/physconst.meta
M src/data/registry.xml
A src/dynamics/utils/hycoef.meta
M src/dynamics/utils/hycoef.F90
A tools/generate_input_to_stdnames_update.py
M tools/inputnames_to_stdnames.py
M tools/stdnames_to_inputnames_dictionary.xml
M cime_config/testdefs/testlist_cam.xml
D cime_config/testdefs/testmods_dirs/cam/outfrq_held_suarez_izumi_nooutput/shell_commands
D cime_config/testdefs/testmods_dirs/cam/outfrq_held_suarez_izumi_nooutput/user_nl_cam
D cime_config/testdefs/testmods_dirs/cam/outfrq_kessler_izumi_nooutput/shell_commands
D cime_config/testdefs/testmods_dirs/cam/outfrq_kessler_izumi_nooutput/user_nl_cam
A cime_config/testdefs/testmods_dirs/cam/outfrq_tj2016_derecho_nooutput/shell_commands
A cime_config/testdefs/testmods_dirs/cam/outfrq_tj2016_derecho_nooutput/user_nl_cam
If there were any failures reported from running test_driver.sh on any test
platform, and checkin with these failures has been OK'd by the gatekeeper,
then copy the lines from the td.*.status files for the failed tests to the
appropriate machine below. All failed tests must be justified.
derecho/intel/aux_cam
derecho/gnu/aux_cam
Summarize any changes to answers, i.e.,