Lightning threat indexes#42
Conversation
grantfirl
left a comment
There was a problem hiding this comment.
The code changes look good. I'll approve once standard name changes from NOAA-EMC/ufsatm#628 have been addressed and the w controversy is settled.
|
@grantfirl - I think we've settled on using w, since there isn't any other way to get the unsmoothed non-hydrostatic vertical wind. Also, I adjusted names based on your requirements. Could you please re-review, and let me know if everything is okay now? |
|
@SamuelTrahanNOAA @welewis69 It is still not clear to me if the unsmoothed w is the best choice since all physics schemes use smoothed w including the microphysics schemes. |
For anything that may eventually affect the prognostic variables, you have to used the smoothed vertical velocity (vvl) so that the smoothing can be used to resolve model stability issues. Also, we must be consistent among physics schemes whose output feeds back into the prognostic variables, otherwise you'll get spurious results when one thinks there's an updraft and another thinks there's nothing. This diagnostic does not feed back into the prognostic variables, ever, and it does need the highest possible precision of w. Smoothing will eliminate the sharp peaks of high w values that are needed to simulate lightning. |
|
@SamuelTrahanNOAA Thanks for the interpretation. It is good to use unsmoothed w then. |
It may also be good to document somewhere that people should not use the unsmoothed vertical velocity unless they're generating purely diagnostic fields. I hope that putting "unsmoothed" in the name will be enough for that. |
|
@grantfirl @dustinswales @Qingfu-Liu All tests are done on ufs-community/ufs-weather-model#1642. Please, go ahead to merge this pr. |
This is part of a collection of PRs to add lightning threat indexes to the RRFS. The algorithm was originally ported from HRRR to the GFDL_atmos_cubed_sphere NGGPS diagnostics, using the w variable in both implementations. I've moved it to CCPP's maximum hourly diagnostics, but it still needs w in the formulation, so I've added the dynamical core's
wvariable aswgrs.Issue is here: ufs-community/ufs-weather-model#1632
This work originates from @welewis69 at University of Wisconsin-Madison, who put it in the NGGPS diagnostics. I moved it to CCPP for this PR.