-
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
Restrict UPP computation from undefined points #306
Restrict UPP computation from undefined points #306
Conversation
Thanks for confirming. We will work on further changes based on your suggestions in another PR later. |
…ion from undefined points
@JesseMeng-NOAA From my tests, the following applications have changed results, please double checking to ensure all changes are expected: fv3lam-- [Wen.Meng@v71a1 ptmp]$ cat fv3r_2019062000_pe_test/NATLEV06.tm00.diff hafs-- hrrr-- rap-- [Wen.Meng@v71a1 ptmp]$ cat rap_2020072316_pe_test/WRFNAT.GrbF16.diff |
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.
Wen and Hui-Ya, looks like at this point neither CZEN and CZMEAN were ever calculated so both remain spval.
then calculation below
FACTRS = CZEN(I,J)/CZMEAN(I,J), so FACTRS=1.0, and
GRID1(I,J)=RSWIN(I,J)*FACTRS, which makes no adjustment
but if we validate CZEN and CZMEAN, GRID1 will be spval.
The FV3 based model don't output czen and czmean. These two variables are assigned spval in read interface. I would think checking czen and czmean's spval values can be skipped. @HuiyaChuang-NOAA may chime in. |
Agree. I will check RSWIN instead. |
Looking into it now. |
@JesseMeng-NOAA Could you sync your branch with upstream/dvelop? |
done |
@JesseMeng-NOAA Please do the final checking per @HuiyaChuang-NOAA and @BoCui-NOAA 's comments, take appropriate actions (keep or further mods) and mark comments as solved. Thanks! |
Final check done. All output checked.
Resolved. |
@JesseMeng-NOAA Please take actions in the comments MISCLN.F (either keep or further modifications) and mark as "conversation solved". |
done |
Commit for issue #240 restrict post computation from undefined points
Results from regression test, due to modification in MDLFLD.f blocks of IGET(389) and IGET(390)
Differences are shown only in the lateral boundaries of fv3r domain
fv3r_2019062000/PRSLEV06.tm00.diff
49:78715958:UGRD:planetary boundary layer:rpn_corr=0.99802:rpn_rms=0.328868
50:80993255:VGRD:planetary boundary layer:rpn_corr=0.999444:rpn_rms=0.209165