-
Notifications
You must be signed in to change notification settings - Fork 104
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
Set Up UPP control files for SFS development #862
Conversation
KarinaAsmar-NOAA
commented
Jan 25, 2024
•
edited by WenMeng-NOAA
Loading
edited by WenMeng-NOAA
- Adds SFS control files that generate products from initial version of SFS list (388 fields). Follows from UPP issue #845
- Remove the hard-wired 12-hour bucket for after F240
@WenMeng-NOAA Please see questions below regarding some of the CPC requested variables: 200-hPa velocity potential: UPP currently does not produce this variable, but it is in the NCEP GRIB2 tables as VPOT (Discipline 0, Category 2, Parameter 5). Since this addition requires modifying post_avblflds.xml, should it be included in this PR? subsurface runoff ("drain" in Noah-MP Land Surface Model: The closest variable found in UPP is an accumulated value (ID 122: Accumulated storm surface runoff). Is this the appropriate field? Total sky downward direct radiation at the surface: UPP does not show any variables of this sort. NCEP GRIB2 Table 4.2-0-4 has some options for downward radiation. Would any of those qualify for this request? The remaining CPC requested variables are in UPP and are being added to the control files: IDs 20 (vertical velocity), 86 (streamfunction), and 162 (helicity). |
The following CPC requested variables have been added to the control files and tested: VVEL (id 20), STRM (id 86), HLCY (162). Hera WCOSS2 |
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.
@KarinaAsmar-NOAA This log file shouldn't be committed.
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.
@WenMeng-NOAA The log file has been deleted.
e0044b1
to
9d9ea33
Compare
02ec2fa
to
562d1de
Compare
562d1de
to
f7525f7
Compare
@WenMeng-NOAA The latest standalone and regression tests with the reverted changes (i.e., removed CPC variables) are in the same paths above. |
@KarinaAsmar-NOAA Please sync your branch with the latest UPP develop. |
@WenMeng-NOAA The branch should be synced now. |
@FernandoAndrade-NOAA You may start the UPP RTs on Hera and Orion. There will no baseline changes expected with this PR. |
The UPP RTs were completed on WCOSS2 without changed results. |
Hera and hercules RTs completed without changes to results, Orion is rerunning fv3r due to a fatal error opening PRSLEV within that test case. |
All RTs across Hera, Orion, and Hercules have finished with no changes to results. |
This PR is ready for merging. |