Skip to content
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

dtc/hwrf-physics: HWRF RRTMG (based on #412) #430

Merged

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Apr 8, 2020

This PR is based on #412 by @mzhangw, but for branch dtc/hwrf-physics instead of dtc/develop. It also contains several bug fixes and updates to the original PR.

The following commits were cherry-picked from #412 (all from @mzhangw):
9e9222a
4d9e68f
6f9fec9
57873f2
7e492ca
298d1ae
5404462

Commit 298d1ae, "remove the connection of iovrlw/iovrsw with physparam", was reverted by commit 28d1bc2, "Clean up HWRF RRTMG additions", since these changes were not required and introduced an unnecessary, large amount of modifications to the radiation codes.

Associated PRs:
#430
NCAR/fv3atm#38
NCAR/ufs-weather-model#36

For regression testing information, see NCAR/ufs-weather-model#36

Copy link
Collaborator

@mzhangw mzhangw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. Thanks for your hard work, Dom!

@climbfuji
Copy link
Collaborator Author

looks good to me. Thanks for your hard work, Dom!

Thanks for reviewing. I am trying to make one more improvement in the ufs-weather-model repo, consolidating the run scripts and namelist templates. These will be tested in the final round of regression tests against the new baseline. Otherwise I think these PRs look good now.

@ligiabernardet
Copy link
Collaborator

I am not clear on how one switches between the GFS and HWRF radiation schemes. Is there a new scheme that corresponds to the HWRF radiation sw/lw?

@mzhangw
Copy link
Collaborator

mzhangw commented Apr 8, 2020 via email

@climbfuji
Copy link
Collaborator Author

climbfuji commented Apr 8, 2020 via email

@ligiabernardet
Copy link
Collaborator

ligiabernardet commented Apr 8, 2020 via email

@climbfuji
Copy link
Collaborator Author

iovr changes the algorithm for cloud overlap. icloud changes the algorithm for calculation of partial clouds. I understand that part. Now, regarding the actual radiative transfer model, are there no differences between HWRF and GFS? On Wed, Apr 8, 2020 at 3:18 PM Dom Heinzeller [email protected] wrote:

@mzhangw will be able to answer this question.

@mzhangw
Copy link
Collaborator

mzhangw commented Apr 8, 2020 via email

@grantfirl
Copy link
Collaborator

I had a similar question as Ligia's. I was under the impression that we agreed to make a different version of the RRTMG schemes for the HWRF suite, but this work is merged in with the existing GFS version. Are we 100% certain that the underlying radiative transfer is IDENTICAL between HWRF and GFS? Are there included version numbers or some other clue that this is indeed the case? In other words, if one were to substitute the actual radiative transfer routines from WRF into the GFS and ran with them, would the result be bit-for-bit?

Copy link
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a problem from a coding point of view, so approved. The programmatic question may still be an issue (did we do what we agreed to do?).

@climbfuji climbfuji merged commit 6c27ff2 into NCAR:dtc/hwrf-physics Apr 9, 2020
@climbfuji climbfuji mentioned this pull request Apr 9, 2020
@climbfuji climbfuji deleted the dtc_hwrf-physics_add_rrtmg branch June 27, 2022 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants