-
Notifications
You must be signed in to change notification settings - Fork 34
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
More obs ... #887
More obs ... #887
Conversation
guillaumevernieres
commented
Jan 29, 2024
- fixes More marine obs as part of the default list #885
It's a lot of copy/pasting. I wonder if we should have an application that generates template yaml files for these sst retrieval ... humm. |
Automated Global-Workflow GDASApp Testing Results:
|
How often are we going to be doing this? |
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.
looks right
5b0b4ec
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.
Thanks for the changes @guillaumevernieres.
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.
Looks right-
b814dd0
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.
So no window for the SSTs?
Yeah ... HELL NO @AndrewEichmann-NOAA : |
@guillaumevernieres , this PR modified the submodule hashes for the following JEDI repos
The date is in Here are the hashes we had in GDASApp
GDASApp at Why do we need to move the submodule hashes back in time with PR #887? |
We don't @RussTreadon-NOAA , my mistake ... Although I see 3 green checks on the review side! I'll submit a PR reverting to what it was before. |