-
Notifications
You must be signed in to change notification settings - Fork 34
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
GF radar reflectivity update for RRFS realtime runs #103
Changes from 1 commit
ab84c01
cfd8485
7a8f693
ab4d5f1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
grantfirl marked this conversation as resolved.
Show resolved
Hide resolved
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -66,20 +66,21 @@ subroutine cu_gf_driver_post_run (im, km, t, q, prevst, prevsq, cactiv, cactiv_m | |
conv_act_m(i)=0.0 | ||
endif | ||
! reflectivity parameterization for parameterized convection (reference:Unipost MDLFLD.f) | ||
if(sqrt(garea(i)).lt.6500.)then | ||
!if(sqrt(garea(i)).lt.6500.)then | ||
ze = 0.0 | ||
grantfirl marked this conversation as resolved.
Show resolved
Hide resolved
|
||
ze_conv = 0.0 | ||
dbz_sum = 0.0 | ||
cuprate = raincv(i) * 3600.0 / dt ! cu precip rate (mm/h) | ||
ze_conv = 300.0 * cuprate**1.4 | ||
if (maxupmf(i).gt.0.05) then | ||
cuprate = max(0.1,1.e3*raincv(i) * 3600.0 / dt) ! cu precip rate (mm/h) | ||
if(cuprate .lt. 0.05) cuprate=0. | ||
ze_conv = 300.0 * cuprate**1.5 | ||
if (maxupmf(i).gt.0.1 .and. cuprate.gt.0.) then | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is not cuprate always >0 after adjustment? Any reason to change cuprate and add the condition of cuprate>0? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
In line 73, If the convective precipitation rate is very weak (<0.05 mm/h), the cuprate is reset to 0. Thus in the if condition of line 75, we exclude the case with weak convective updraft and weak convective precipitation. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The problem is in line 72, cuprate will be always >0.1 mm/h. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for catching that. Yes, we have duplicated safeguards. I will update line 72 just as "cuprate = 1.e3*raincv(i) * 3600.0 / dt" in this PR soon. |
||
do k = 1, km | ||
ze = 10._kind_phys ** (0.1 * refl_10cm(i,k)) | ||
dbz_sum = max(dbzmin, 10.0 * log10(ze + ze_conv)) | ||
refl_10cm(i,k) = dbz_sum | ||
enddo | ||
endif | ||
endif | ||
!endif | ||
enddo | ||
grantfirl marked this conversation as resolved.
Show resolved
Hide resolved
|
||
!$acc end kernels | ||
|
||
|
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.
I would suggest to just remove these lines.
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.
I suggest we keep this just in case we need to tune with subcloud evaporation in the future.
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.
We've been recommending that folks remove code rather than comment it out because folks often forget why they left commented code in and it makes the code harder to read in the long run. Git saves the history, so one can always go back to a commit before this PR and put these lines back in.
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.
Ok, I will remove these commented lines.