-
Notifications
You must be signed in to change notification settings - Fork 30
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 SNOCVR snow depth python api code to GDASApp #848
Conversation
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.
Just one comment to change 2023 -> 2024.
I asked about snow depth being mm but that's what the user wants, so this is fine with me.
I also asked about units in the argo file being "below" the surface, and got confirmation that the data is indeed below the surface.
The rest looks good. Approved.
Thank you, Nick. Changed as suggested. |
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.
Remove the argo python script update.
The code changes look good
The ctest "test_gdasapp_setup_cycled_exp" is failing with the automatic github check. However, if I run it on my own Orion machine with my modules, it does not fail. What could be the cause of this? @emilyhcliu @RussTreadon-NOAA These are the modules I use:
I only get a warning when I run from Orion:
|
Nick just helped me to do such a test. it passed ctest via ctest -VV -R test_gdasapp_setup_cycled_exp in my build directory. 100% tests passed, 0 tests failed out of 1 Total Test time (real) = 8.57 sec |
@CoryMartin-NOAA, I talked this problem with @emilyhcliu, and lets her see what happened. This issue is not related to my work. I only added one json configure file and one python snowcvr file without changing any other parts. Although I can recreate a feature branch to add these two files, and create a new PR if needed. Could you please give me some instructions and guidance. Thank you |
@YoulongXia-NOAA my guess is you just needed to merge develop in, let me start the automated tests and see if it passes, if so, I'll approve and merge |
@thank you, @CoryMartin-NOAA for your help. Next time I will recall to merge develop to my feature and then git push. |
Automated Global-Workflow GDASApp Testing Results:
|
this looks great, @CoryMartin-NOAA what do you think about either a fixed yaml or python in a sense demonstration yaml with the set key values from the json are set as defaults? will re-read but believe that is doable dyt |
yes that seems reasonable, and just noting that many of these python scripts will probably change as we go to reading directly rather than converting to intermediate IODA files |
would make the most sense. think just having demonstration capability is all that is needed, researchers, students, there are going to be cases where people are populating archives there the windowing information, receipt time, characteristics need to be well traced |
This PR is mainly to add snocvr bufr dump snow depth data using python API to replace missing WGOSLID with local station ID to support Land DA task. The snocvr snow depth is new operation product and it did not use before. The dataset includes snow depth from several European countries and US abd Canada (mainly comes from MADIS).