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

v12: Updates and fixes for SCM #662

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Nov 8, 2024

This is a PR with fixes for the SCM in v12.

The changes are:

  1. Use the git-v12.0 scm inputfiles which moves to GFDL microphysics by default
  2. Move to 181 levels by default
  3. Set some new sed bits
  4. Copy over fvcore_layout.rc into input.nml

I'm keeping this draft until @narnold1 can test. It works for me in one case at least. If successful for him, we can merge and I can try it out in my nightly testing suite which runs all the cases

@mathomp4 mathomp4 added Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs Non 0-diff The changes in this pull request are non-zero-diff and removed Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs labels Nov 8, 2024
@mathomp4 mathomp4 marked this pull request as ready for review November 8, 2024 20:26
@mathomp4 mathomp4 requested a review from a team as a code owner November 8, 2024 20:26
@mathomp4 mathomp4 requested a review from narnold1 November 8, 2024 20:29
@mathomp4 mathomp4 marked this pull request as draft November 8, 2024 20:29
@mathomp4 mathomp4 marked this pull request as ready for review December 2, 2024 19:09
@sdrabenh sdrabenh merged commit 86a2fcb into feature/sdrabenh/gcm_v12 Dec 13, 2024
19 of 24 checks passed
@mathomp4 mathomp4 deleted the bugfix/mathomp4/scm-fix branch December 13, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non 0-diff The changes in this pull request are non-zero-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants