-
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
Add the capability to assimilate the MADIS snow depth data from GTS #1836
Merged
Merged
Changes from 9 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
3a723e8
Initialize the branch to do the madis DA.
jiaruidong2017 46577a1
Merge branch 'develop' into feature/madis_da
jiaruidong2017 32e24b4
Merge branch 'develop' into feature/madis_da
jiaruidong2017 7f44dd4
Remove some repetitions.
jiaruidong2017 0273248
Fix the python norm error.
jiaruidong2017 4d3e105
Simplify the code to generate bufr2ioda YAML files
jiaruidong2017 9bce693
Remove the obsolete BUFR2IODAYAML and BUFRADPSFCYAML.
jiaruidong2017 0017549
Merge branch 'develop' into feature/madis_da
jiaruidong2017 60567df
Address reviewer's comments.
jiaruidong2017 dc83591
Simplify the codes following the reviewer's suggestions.
jiaruidong2017 94a0ff8
Remove the unnecessary construction of gts_yaml.
jiaruidong2017 04f7ec5
Merge branch 'develop' into feature/madis_da
jiaruidong2017 73fcc1c
Update the GDASApp's commit hash.
jiaruidong2017 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
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.
Thanks for updating this section @jiaruidong2017
Here is what I was suggesting.
In the attached patch, you will notice that I have removed any knowledge of what is being converted from bufr to ioda in the python script. The control comes directly from the yaml.
Would you be willing to give this a try?
diff.txt
Thanks
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 @aerorahul. I will try it and let you know then.
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.
@aerorahul I just tested and it worked perfectly. The changes have been committed to the repo.