-
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
base: develop
Are you sure you want to change the base?
Allow APP to differ between RUNs #2943
Conversation
This is going to require some more leg work as some APP-specific flags are currently read in without specifying a global-workflow/workflow/applications/applications.py Lines 34 to 50 in c6e3262
This will require reworking the |
217ec77
to
8275caf
Compare
This PR now correctly generates identical CI XMLs as compared to develop (with the exception of the Marking ready for review. |
Hercules was already correct Refs NOAA-EMC#2956
Experiment C96C48_hybatmDA FAILED on Orion in Build# 1 in |
Experiment C96_atm3DVar FAILED on Orion in Build# 1 in |
|
Thanks @TerrenceMcGuinness-NOAA. I guess I need to readjust |
Experiment C48_ATM FAILED on Orion in Build# 1 with error logs:
Follow link here to view the contents of the above file(s): (link) |
Experiment C48_ATM FAILED on Orion in Build# 1 in |
Experiment C96_S2SWA_gefs_replay_ics FAILED on Orion in Build# 1 in |
CI Failed on Orion in Build# 1
|
4489259
to
76a62d0
Compare
I'm having some issues with this branch after merging the interval PR. Converting to draft until I get it squared away. |
All features are working again. Reopening this PR for review. |
Description
This enables APP to be specified for each RUN. This also removes the need for a
_no_run
configuration dictionary and somewhat simplifies the_init_finalize
method.Resolves #2908
Resolves #2956
Type of change
Change characteristics
How has this been tested?
{"gdas": "S2S", "gfs": "ATM"}
)Checklist