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

CPLHIST compsets should use same land tunings as for CAM compsets #1885

Closed
billsacks opened this issue Oct 27, 2022 · 0 comments
Closed

CPLHIST compsets should use same land tunings as for CAM compsets #1885

billsacks opened this issue Oct 27, 2022 · 0 comments
Assignees
Labels
bug something is working incorrectly

Comments

@billsacks
Copy link
Member

Brief summary of bug

Compsets using DATM%CPLHIST mode should use the same tunings used for runs coupled to CAM, but instead have been using CRUv7 tunings.

General bug information

CTSM version you are using: master

Does this bug cause significantly incorrect results in the model's science? Yes, but only for CPLHIST cases

Configurations affected: Runs with DATM%CPLHIST mode

Details of bug

My thinking - which others at today's ctsm-software meeting support - is that CPLHIST cases are typically generated using CAM output, and so should use the same tunings as are used in actual coupled CAM runs. @olyson says that for the land spinups done for CESM2 / CMIP6 he changed the tuning mode to use these CAM tunings, so those runs were fine, but we want the out-of-the-box behavior to match what he did manually for those runs.

I will change this using a regular expression like DATM%CPLHIST.*_CLM50. The use of .* following CPLHIST will allow matching the newly-proposed CPLHISTCMEPS (or something like that).

@billsacks billsacks added the bug something is working incorrectly label Oct 27, 2022
@billsacks billsacks self-assigned this Oct 27, 2022
billsacks added a commit that referenced this issue Oct 28, 2022
Fix some compsets; add only clauses for ESMF use statements

(1) Fix I1850Clm51BgcCrop compset (was using CLM50 instead of CLM51)
- Resolves #1882

(2) Change LND_TUNING_MODE for DATM%CPLHIST compsets to use CAM tunings
    since these cases typically use atmosphere forcings from CAM.
- Resolves #1885

(3) Add "only" clauses to ESMF use statements
- Resolves #1846
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly
Projects
None yet
Development

No branches or pull requests

1 participant