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

Cleanup GFS_layer and CMakeLists for GMTB-SCM (ccpp-physics) #109

Conversation

climbfuji
Copy link
Collaborator

This PR removes the GFS_layer directory and places GFS_suite_setup_scm.F90 in physics (GFS_typedefs.F90 is moved to the gmtb-scm repository where it belongs).

The PR also replaces the hardcoded list of schemes and caps for SCM with the auto-generated cmake include files and performs cleanup work in the various CMakeLists.txt files.

This PR goes together with the two PRs listed below.

Changes tested for bit-for-bit reproducibility with SCM and NEMSfv3gfs (although changes do not affect NEMSfv3gfs, but to be on the safe side) on MacOSX/GNU.

…CCPP-SCM (newly defined in SCM's CMakeLists.txt, until now assumed as default = project 'Unknown'), replace explicit list of schemes and caps with auto-generated versions
@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 climbfuji merged commit c0c9a35 into NCAR:master Jul 2, 2018
@climbfuji climbfuji deleted the gmtb-scm-cleanup-gfs-layer-and-cmakelists-20180629 branch July 2, 2018 15:20
climbfuji pushed a commit to climbfuji/ccpp-physics that referenced this pull request Oct 25, 2021
…in_20211019

Update gsl/develop from main 2021/10/19
dustinswales pushed a commit to dustinswales/ccpp-physics that referenced this pull request Oct 18, 2023
passing nc back from microphysics
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