-
Notifications
You must be signed in to change notification settings - Fork 180
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
Add initial GSI-based soil analysis capability #2263
Merged
WalterKolczynski-NOAA
merged 24 commits into
NOAA-EMC:develop
from
ClaraDraper-NOAA:feature/soil_updates
Mar 18, 2024
Merged
Changes from 13 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
6438626
First pass at adding the GSI-based soil analysis to the workflow.
ClaraDraper-NOAA f9d0442
Cleaned up print statememts. Runs, adds increments to ensemble only.
ClaraDraper-NOAA 3ab6e71
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA d1434bb
Clean-up, ready for PR.
ClaraDraper-NOAA 7e0e225
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA 3f507c0
minor updates for git's spell-check.
ClaraDraper-NOAA 528a511
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA c767ca6
removed work-arounds to avoid call to global_cycle/sfccycle with
ClaraDraper-NOAA 2815023
Updated gsi hash to catch soil analysis updates.
ClaraDraper-NOAA fb4917a
Turn soil analysis off for default.
ClaraDraper-NOAA 1b0c4a0
Merge branch 'feature/soil_updates' of https://github.com/ClaraDraper…
ClaraDraper-NOAA 9f1d772
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA 13026e3
bug fixes.
ClaraDraper-NOAA 734ed4e
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA f77d91f
Update UFS_UTILS hash.
ClaraDraper-NOAA 7168006
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA 0dcc2c4
bug fix.
ClaraDraper-NOAA 2129fac
Clean up merge, resolve spelling issues.
ClaraDraper-NOAA 2b6694e
Updated the gsi fix version.
ClaraDraper-NOAA 597b6e0
Merge branch 'develop' into feature/soil_updates
ClaraDraper-NOAA 7e78514
Fixed merge commit in defaults.yaml
ClaraDraper-NOAA 2fb2e73
Merge branch 'develop' into feature/soil_updates
aerorahul 38cf220
Merge branch 'develop' into feature/soil_updates
aerorahul 98660b1
Merge branch 'develop' into feature/soil_updates
aerorahul File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
Submodule gsi_enkf.fd
updated
143 files
Oops, something went wrong.
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.
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.
Can we make this a templated variable and set it to
NO
as the default?That way, we can set this to
YES
in one of the existing tests and ensure this capability is tested regularly.The test this could piggy back off on is C96C48_hybatmDA.yaml
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.
Rahul - I'm not familiar with the templating, but followed through the example for DOIAU. Is this all I need to do?
In config.base.emc.dyn change the line highlighted above to:
export GSI_SOILANAL=@GSI_SOILANAL@
Then in parm/config/gfs/yaml/defaults.yaml I set the default to NO.
For the test, should I add GSI_SOILANAL=YES to C96C48_hybatmDA.yaml, or create a new test based on that one?