-
Notifications
You must be signed in to change notification settings - Fork 102
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
MPAS applications: fixes for absolute vorticity and surface latent-heat flux #1069
Conversation
@gspetro-NOAA @FernandoAndrade-NOAA You may start the UPP RTs on R&D machines. There should be no baseline changes with this PR. |
The UPP RTs have been completed on WCOSS2 with no baseline changes. |
@jaymes-kenyon Please take a look at my UPP standalone test with MPAS_hfip data at
|
@WenMeng-NOAA — Your test looks good—thanks for sharing! |
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.
RTs passed on Hera, Orion, and Hercules with no baseline changes.
This PR is ready for merging. |
This PR addresses issue #1068. It has been tested in MPAS applications (GSL's RRFS and HFIP configurations).
(1) Fix for absolute vorticity: Currently, in INITPOST_MPAS.F, the U and V winds are interpolated (averaged) onto mass points without checking for missing values. However, for the HFIP domain, a region of missing values surrounds the computational domain. As a result, the UPP interpolation procedure at the domain edge is contaminated by adjacent "SPVAL" grid points. Moreover, the subsequent calculations involving horizontal U and V gradients (e.g., for absolute vorticity, or "ABSV") yield very large and "noisy" values at the domain edge. The GRIB2 compression struggles with these "ABSV" fields; see attached screenshot of 850-hPa absolute vorticity (left panel).
This PR ensures that U and V winds are interpolated onto mass points only after checking for the presence of "SPVAL" at adjacent grids. Additionally, the "ABSV" arrays are initialized with "SPVAL", regardless of MODELNAME. The corrected field of 850-hPa absolute vorticity is shown in the attached screenshot (right panel).
(2) Fix for surface latent-heat flux: This field is now read-in from the netCDF "LH" array