Conversation
…yet. FATES code is still having problems like not getting the shr_log_mod and errMsg subroutine in it, and this is setup for the shr_mpi_mod stub, but the testing will need to use the full shr_mpi_mod code. It also looks like getting the CESM unit testing to work with MPI would be an effort as we don't even build pFUnit with MPI libraries (although we did on Cheyenne)
Conflicts: src/main/test/CMakeLists.txt
…ds working, had to remove decompInitMod and subgridMod
…ng the env variable MPISERIAL to \/glade/u/apps/derecho/23.09/spack/opt/spack/mpi-serial/2.5.0/oneapi/2023.2.1/p3fw and it appears that you have to run the build twice for it to work, as such I'll revert this afterwards. But, I'm saving the commit so it could be done easily in the future
…es setting the env variable MPISERIAL to \/glade/u/apps/derecho/23.09/spack/opt/spack/mpi-serial/2.5.0/oneapi/2023.2.1/p3fw and it appears that you have to run the build twice for it to work, as such I'll revert this afterwards. But, I'm saving the commit so it could be done easily in the future" This reverts commit 0461ea5.
ekluzek
commented
Jan 20, 2026
ekluzek
commented
Jan 20, 2026
slevis-lmwg
approved these changes
Jan 21, 2026
Contributor
slevis-lmwg
left a comment
There was a problem hiding this comment.
Thank you @ekluzek :-)
Co-authored-by: Samuel Levis <slevis@ucar.edu>
Correction from review. Co-authored-by: Samuel Levis <slevis@ucar.edu>
…irst time it's built, you have to build it twice for it to work. I commented it out and added some notes around it
slevis-lmwg
added a commit
to slevis-lmwg/ctsm
that referenced
this pull request
Jan 23, 2026
Merge b4b-dev to master - Change some _U and _R history fields to be on by default PR ESCOMP#3667 by Keith Oleson - Update to cime version that fixes the check_input_data --download issue PR ESCOMP#3647 by Erik Kluzek; ESCOMP#3647 updated to cime6.1.145, while updating b4b-dev to master in this PR gets us to cime6.1.146. - Initial fixes to generate_gdd20_baseline PR ESCOMP#3543 - Decomp mod unittest PR ESCOMP#3699
slevis-lmwg
added a commit
to slevis-lmwg/ctsm
that referenced
this pull request
Jan 23, 2026
Merge b4b-dev to master - Change some _U and _R history fields to be on by default PR ESCOMP#3667 by Keith Oleson - Update to cime version that fixes the check_input_data --download issue PR ESCOMP#3647 by Erik Kluzek; ESCOMP#3647 updated to cime6.1.145, while updating b4b-dev to master in this PR gets us to cime6.1.146. - Initial fixes to generate_gdd20_baseline PR ESCOMP#3543 - Decomp mod unittest PR ESCOMP#3699
glemieux
added a commit
to glemieux/ctsm
that referenced
this pull request
Jan 29, 2026
Merge b4b-dev to master - Change some _U and _R history fields to be on by default PR ESCOMP#3667 by Keith Oleson - Update to cime version that fixes the check_input_data --download issue PR ESCOMP#3647 by Erik Kluzek; ESCOMP#3647 updated to cime6.1.145, while updating b4b-dev to master in this PR gets us to cime6.1.146. - Initial fixes to generate_gdd20_baseline PR ESCOMP#3543 - Decomp mod unittest PR ESCOMP#3699
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of changes
This adds a unit test for decompMod, that I need for #3666
Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #):
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Does this create a need to change or add documentation? Did you do so? No No
Testing performed, if any: So far just unit tests
(since this doesn't change model code there doesn't seem a need to run aux_clm)