-
Notifications
You must be signed in to change notification settings - Fork 168
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
Allow APP to differ between RUNs #2943
Open
DavidHuber-NOAA
wants to merge
45
commits into
NOAA-EMC:develop
Choose a base branch
from
DavidHuber-NOAA:feature/multi-app
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
c0102bf
Additions to allow APP to differ by RUN
DavidHuber-NOAA d987c38
Make cleanup and stage jobs run on service partitions
DavidHuber-NOAA 599beeb
Combine get_valid_runs and get_task_names
DavidHuber-NOAA eaa1049
Simplify get_task_names
DavidHuber-NOAA 3787be7
Make APP-specific options based on RUN
DavidHuber-NOAA a4ab75c
Move _get_app_configs invokation to _source_configs
DavidHuber-NOAA 4ae0f36
Make the Application classes run-configurable
DavidHuber-NOAA 4374bad
Fix f-string formating
DavidHuber-NOAA e6b1920
Application bug fixes
DavidHuber-NOAA d94b685
Update app_config options in the Tasks classes
DavidHuber-NOAA 10763cc
Fix hybvar ocn analysis task tabbing
DavidHuber-NOAA 2f07ddb
Merge develop branch
DavidHuber-NOAA 7dbf11a
do_extractvars bug fix
DavidHuber-NOAA de57402
Fix do_extractvars bug, remove unused modules
DavidHuber-NOAA 7253eef
Fix additional do_* bugs
DavidHuber-NOAA 9e96844
Rearrange gfs_cycled task order
DavidHuber-NOAA 8275caf
Rearrange XML RUN order to match develop
DavidHuber-NOAA 60f6384
Merge develop, address conflicts in gfs_cycled and gfs_tasks
DavidHuber-NOAA 4477e60
Fix merge bugs
DavidHuber-NOAA e8a9b96
Increase max allocatable memory on Orion
DavidHuber-NOAA e4ff015
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 71a2c4e
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 731f16e
simplify aerosol/wave RUN-based options
DavidHuber-NOAA 4263828
Merged develop, address conflicts
DavidHuber-NOAA d829a3a
Remove unreferenced stage_ic->prep_emissions dependency
DavidHuber-NOAA f0a99c6
Remove unused wave_runs variable
DavidHuber-NOAA 917b9ed
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 6fe6014
Replace references to DO_AERO with DO_AERO_FCST
DavidHuber-NOAA 889daf7
Remove redundant RUN-case block
DavidHuber-NOAA 2e92009
Add APP initialization back in
DavidHuber-NOAA daecdc9
Move NET/MODE options to _get_run_options
DavidHuber-NOAA c80def6
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 1975b2f
Reenable GEFS aerosol forecasts
DavidHuber-NOAA 506709e
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 76a62d0
Reduce Orion available memory
DavidHuber-NOAA 8a255cc
Merge in develop
DavidHuber-NOAA 69fb251
Merge remote-tracking branch 'origin/develop' into feature/multi-app
DavidHuber-NOAA 2d4139b
Fix interval_gfs conflicts
DavidHuber-NOAA b342a67
Resolve additional interval_gfs conflicts
DavidHuber-NOAA da49bdb
Merge branch 'develop' into feature/multi-app
DavidHuber-NOAA 5778ed5
Merge branch 'develop' into feature/multi-app
DavidHuber-NOAA 2e25776
Merge branch 'develop' into feature/multi-app
DavidHuber-NOAA d2f9121
Merge remote-tracking branch 'emc/develop' into feature/multi-app
DavidHuber-NOAA 8d6e7f0
Merge branch 'develop' into feature/multi-app
DavidHuber-NOAA ce03a22
Merge branch 'develop' into feature/multi-app
DavidHuber-NOAA 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
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
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
Oops, something went wrong.
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.
Why is there no
DO_AERO
for GEFS anymore (and not exported for GFS)?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.
Because aerosols are controlled at the forecast and analysis level, not the application level.
For instance, previously, this was implemented in the
config.fcst
config by reading theAERO_FCST_RUN
variable:global-workflow/parm/config/gfs/config.fcst
Lines 11 to 21 in 152bb45
But now, it can rely on just the
DO_AERO_FCST
variable.Keeping the
DO_AERO
flag would be redundant since it is superseded byDO_AERO_FCST
(orDO_AERO_ANL
in the analysis jobs).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.
I debated whether to add
DO_AERO
(without exporting) toconfig.gefs
. When analyses are enabled, having the flag will be handy.