-
Notifications
You must be signed in to change notification settings - Fork 22
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
KeyClim Cloud2 tunings #142
KeyClim Cloud2 tunings #142
Conversation
Defaults are 268.15 and 238.15 NorESM defaults are 243.15 and 238.15
…mp and clubb_dt_low
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.
Hi Steve, seems fine for me. Best regards, Dirk
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.
The changes look fine. One question: The purely cosmetic changes in some of the files, typically removal of empty characters at the end of the line. Is that on purpose or is it caused by using different editing tools?
I have all my editors set to automatically remove spaces at the end of each line when saving a file. This is the practice we adopted at NCAR to (eventually) lower the number of spurious changes presented in a PR. Note that when reviewing a PR, you can always hide these whitespace changes. Just below the "Files changed" tab is a gear icon. Clicking on that reveals a "Hide whitespace" checkbox which will hide those changes for easier reviewing. I plan to continue this practice as I edit files since many have accumulated a lot of these extra spaces over the years. Let me know if that is a problem or if you would like to also adopt the practice. |
OK. No problems with the practice |
1793884
to
a6e5398
Compare
Summary: Make KeyClim Cloud2 tunings the default for NFHIST and HF1850
Contributors: gold2718, oyvindseland
Reviewers: DirkOlivie, oyvindseland
Purpose of changes: Prepare compsets to use KeyClim Cloud2 tunings as the default
Github PR URL: #142
Changes made to build system: None
Changes made to the namelist: Added new namelist items to allow for new tuning parameters
Changes to the defaults for the boundary datasets: None
Substantial timing or memory changes: None
Test suite aux_cam_noresm runs with the expected namelist and baseline changes
Issues addressed by this PR: Create new compsets for NorESM2.3, NorESMhub/NorESM#465