You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
ekluzek
added
enhancement
new capability or improved behavior of existing capability
priority: high
High priority to fix/merge soon, e.g., because it is a problem in important configurations
labels
Jun 28, 2023
@ekluzek I wonder what a timeline for merging this change could be? I ask because @danicalombardozzi is working on revisions to our NEON paper and assumed that we'll also need @briandobbins to make a new docker image with the code modifications that have evolved since our last NEON branch tag?
Hi Will. Let's talk more tomorrow. But, we've been delaying on this to get other work in. And we want the BGC sequencing work to come in as soon as it's ready so difficult merges won't have to happen. This is simple enough and orthogonal to that, that it's going to be OK to come in before that if needed. I think we should be able to get it in sometime next week.
FATES API update to facilitate fates refactor
This updates a number of FATES type names and module use statements
which correspond with a refactoring effort that moves FATES
patches and cohorts into their own respective modules.
With the FATES update is a minor science update, so there are
changes to answers for FATES.
This also incorporates a minor update to a more recent version
of the ccs config external.
The testing isn't as critical, but the updates to shell commands is the bug fix I'd like to bring in.
cime_config/usermods_dirs/NEON/defaults/shell_commands
enhancementnew capability or improved behavior of existing capabilitypriority: highHigh priority to fix/merge soon, e.g., because it is a problem in important configurations
3 participants
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
More confined regular expression for NEON. And a few simple fixes.
Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #):
Fixes #2039
Fixes #2103
Fixes #2028
Fixes #1506
Fixes #1499
Are answers expected to change (and if so in what way)? Yes? For some NEON cases?
Any User Interface Changes (namelist or namelist defaults changes)? No
Testing performed, if any: None so far, but regular testing will be run