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

Change nextsw_cday calculation for cam_dev (CAM7) in cplhist mode #229

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

olyson
Copy link
Contributor

@olyson olyson commented May 2, 2023

Description of changes

The radiation re-ordering in cam_dev (CAM7) requires a change to the calculation of nextsw_cday in cplhist mode.

Specific notes

Note that this behavior is triggered by the namelist variable cplhist_nextsw_cday_calc which is not yet implemented. More development is needed to incorporate this namelist variable. The name itself may also change.

Contributors other than yourself, if any: @billsacks @adamrher

CDEPS Issues Fixed (include github issue #):

Are there dependencies on other component PRs (if so list):

Are changes expected to change answers (bfb, different to roundoff, more substantial):

Any User Interface Changes (namelist or namelist defaults changes):

Testing performed (e.g. aux_cdeps, CESM prealpha, etc):

Hashes used for testing:

olyson added 2 commits May 2, 2023 10:59
Note that cplhist_nextsw_cday_calc is intended to be a namelist parameter
but is not yet implemented here.
@billsacks billsacks self-requested a review May 4, 2023 22:06
@billsacks billsacks self-assigned this May 4, 2023
@billsacks
Copy link
Member

This is still on my radar, but other priorities have come up that have prevented me from working on this. I still will work on this, but it may not be until after some vacation and the CESM workshop – so in late June or July.

@wwieder
Copy link

wwieder commented Sep 26, 2024

@billsacks it seems like @olyson may need help with namelist changes associated with this PR? If we're going to start running with CPL_HIST with CESM3 output soon, this should likely be addressed.

@billsacks
Copy link
Member

Yes. Sorry for the continued delay on this. It is coming up to the top of my list.

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.

3 participants