Remove LILAC references to mct #2374
Merged
Conversation
This file contains 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
It looks like the need for MCT was removed in b4d64ac, when Mariana changed the stream code to use CDEPS, but the initialization of MCT wasn't removed at that time... but it looks like MCT is now unneeded, so going ahead and removing it now.
jedwards4b
approved these changes
Feb 16, 2024
ekluzek
added
enhancement
|
@billsacks I'm going to bring this in after #2385 so that it will bring in the fix for unit testing on Derecho. Then we'll be able to see the unit tests passing on Derecho. |
|
Unit-testing now PASSes on Derecho with updated branch. This also just resolves one checkbox in #2294. But, I'll bring it in as it is... |
|
Testing ran as expected, so merging. |
Merged
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
Remove remaining references to mct. These appeared in two places:
Specific notes
Contributors other than yourself, if any: @jedwards4b identified these
CTSM Issues Fixed (include github issue #):
Does one checkbox in #2294
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Testing performed, if any:
Note: I ran the following testing with these changes off of ctsm5.1.dev167, then rebased the changes to the b4b-dev branch for the sake of this PR: