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

Merge ctsm5.1.dev169 into hillslope_hydrology #10

Closed

Conversation

samsrabin
Copy link

I've done the aux_clm testing and everything looks good. When you approve this, we should be able to bring it in to the main CTSM branch!

adrifoster and others added 30 commits August 29, 2023 10:20
…r2212

Merge tag: Turn on Meier2022, tillage, residue removal; bring in ESCOMP#2212
samsrabin and others added 29 commits February 10, 2024 11:30
Do not crash "make all" even if pylint isn't clean
Resolve pylint issues in python/ctsm/.
… to derecho_gnu.

derecho_intel doesn't currently work with debug mode on.
…derecho

Move FSURDATMODIFYCTSM test to Derecho
Delete history and spurious _FillValue from parameter files
Do not allow negative snocan in a part of the code that triggers the error reported in issue ESCOMP#2366.
Remove a case of negative snocan in CanopyFluxesMod to resolve ESCOMP#2366

One-line change that now includes "max(0._r8," to prevent negative
snocan in that part of the code.
Brings in 3 PRs from b4b-dev to master:
- Do not crash "make all" even if pylint isn't clean (ESCOMP#2353; Sam Rabin)
- Resolve pylint issues (ESCOMP#2354; Sam Rabin)
- Move FSURDATMODIFYCTSM test to Derecho (ESCOMP#2364; Sam Rabin)
Merge pull request ESCOMP#2382 from samsrabin/merge-b4bdev-20240222

Brings in 3 PRs from b4b-dev to master:
- Do not crash "make all" even if pylint isn't clean (ESCOMP#2353; Sam Rabin)
- Resolve pylint issues (ESCOMP#2354; Sam Rabin)
- Move FSURDATMODIFYCTSM test to Derecho (ESCOMP#2364; Sam Rabin)
@samsrabin samsrabin closed this Feb 28, 2024
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.

4 participants