Skip to content
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

NOAA GSL updates to output new simulated satellite brightness temps #570

Merged
merged 11 commits into from
Oct 19, 2022

Conversation

EricJames-NOAA
Copy link
Contributor

@EricJames-NOAA EricJames-NOAA commented Oct 7, 2022

This PR includes updates to UPP to output new simulated satellite brightness temperatures from the RRFS. The code was developed by Will Lewis from Wisconsin / SSEC, and merged into GSL RRFS_dev branch during 2022. It is now used in realtime RRFS_NA_3km system at GSL. Tests on Jet show identical output for RRFS except for the new fields.

Ka Yee Wong ran UPP regression tests on Hera, and the only non bitwise identical results are in the naming of the soil moisture content variable:

[Eric.James@hfe09 ejames]$ diff sfluxgrbf006_pe.txt sfluxgrbf006_pe_hera.txt
161c161
< 21:47496981:vt=2021021606:0-2 m below ground:6 hour fcst:SOILM Soil Moisture Content [kg/m^2]:

21:47496981:vt=2021021606:0-2 m below ground:6 hour fcst:SOIL_M Soil Moisture [kg/m^3]:

This discrepancy shows up for slightly more than half of the regression tests.

@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA Can you open a UPP issue to link this PR? Thanks!

@WenMeng-NOAA WenMeng-NOAA linked an issue Oct 7, 2022 that may be closed by this pull request
@WenMeng-NOAA WenMeng-NOAA added No Baseline Change No baseline of the UPP regression tests are made. Ready for Review This PR is ready for code review. labels Oct 19, 2022
@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA I ran the UPP standalone test with model data files you provided before and UPP control file from your PR. It appears to me simulated satellite products were generated. My test results are at /scratch1/NCEPDEV/stmp2/Wen.Meng/rrfs_2021100815
Can you take a look at and let me know if they are inconsistent with your test results.

@EricJames-NOAA
Copy link
Contributor Author

@WenMeng-NOAA the results look good to me! Thank you for your test.

@WenMeng-NOAA
Copy link
Collaborator

The UPP RT tests were completed on WCOSS2, Hera and Orion. No baseline update is needed from this PR.

@WenMeng-NOAA WenMeng-NOAA merged commit ac6bfbd into NOAA-EMC:develop Oct 19, 2022
EricJames-NOAA pushed a commit to EricJames-NOAA/UPP that referenced this pull request Oct 18, 2023
Co-authored-by: benjamin blake <benjamin.blake@dlogin05.dogwood.wcoss2.ncep.noaa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Baseline Change No baseline of the UPP regression tests are made. Ready for Review This PR is ready for code review. RRFS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NOAA GSL UPP updates
2 participants