-
Notifications
You must be signed in to change notification settings - Fork 176
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
Fix IC staging bug: should be idatestr
not rdatestr
#1783
Comments
For warm starts, atm data is linked from the previous cycle, so you would want |
While testing this for warm starts, I encountered errors for missing increment files (ensemble @WalterKolczynski-NOAA Is there a reason I would not want to link in the entirety of the analysis/atmos/ directory from the warm start ICs (i.e. calling |
I think we'd rather have an explicit list. That way you only copy what you need and also know if something is missing. @aerorahul? |
There are 3 ways to start the atmosphere:
1 is the way most (99%) of developers start their experiment (generate cold start ICs offline from ops). 2 is also functional if someone wants to start from warm start ICs but no IAU. For 3; I think there are blockers in the forecast scripts that will prevent doing this; i.e. IAU is turned off for the first half cycle (I will need to check to be sure) Is 3 a mode we want to support? Yes. Is it high on the priority list? Probably not worth the effort ATM. |
@aerorahul Noted, thanks. I will continue my warm-start testing with IAU turned off then. |
This fixes a bug in IC linking within setup_expt.py by correcting the date string based on experiment start type -- warm starts should look at `rdate` (-6 hours) and cold starts should look at `idate` (current cycle). Fixes #1783
Originally posted by @CoryMartin-NOAA in #1529 (comment)
The text was updated successfully, but these errors were encountered: