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.
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 observation preparation job for aerosols DA to workflow #2624
Add observation preparation job for aerosols DA to workflow #2624
Changes from all commits
f7ea78c
9d5aeb7
a8faff2
bdc5338
c014ae1
8d64fdd
5c31d4a
1b7482a
c224002
8beebee
68d3e2c
18f11fb
7a631d9
8f78765
25616c4
8bc479a
03324f5
3db2938
8fa0ecd
777437f
bfdfe07
43ec97c
beaed42
d0aa0da
f5bd885
adbe97c
a941154
cac1e3d
75b6d6b
5dd514b
8d0ce42
7f6d171
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Is this right? this means the input OBS would come from the current working directory
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.
Agree.
$DATA
will be empty at the start of the job (and wouldn't be considered aCOM
path).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.
This will likely need to be updated, but that can wait since the new job is now off by default. Can you create an issue for this @CoryMartin-NOAA?
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.
#2691
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.
@ypwang19 can you download the obs for 2021-12-20 18z and 2021-12-21 00z and provide them to @WalterKolczynski-NOAA so that they can be staged in the proper spot for the CI test.
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.
Perhaps this could be wrapped into the new issue.
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.
Does this prep job really need this much memory?
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.
Probably not, but it is somewhat intensive, but this is probably an order of magnitude too large, I can test next week with less memory
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.
This can be addressed later.