Skip to content

RRTMG lw/sw radiation updates from NEMSfv3gfs#50

Merged
grantfirl merged 1 commit into
NCAR:masterfrom
climbfuji:gmtb-scm-radiation-updates-from-nemsfv3gfs
Jun 27, 2018
Merged

RRTMG lw/sw radiation updates from NEMSfv3gfs#50
grantfirl merged 1 commit into
NCAR:masterfrom
climbfuji:gmtb-scm-radiation-updates-from-nemsfv3gfs

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

This PR updates the metadata tables in src/gmtb_scm_type_defs.f90 to run the updated RRTMG lw/sw schemes from NEMSfv3gfs current trunk. This PR requires several PRs listed below to be merged at the same time.

SCM was tested on MacOSX/GNU for the twpice testcase only.

@climbfuji
Copy link
Copy Markdown
Collaborator Author

climbfuji commented Jun 26, 2018

@climbfuji
Copy link
Copy Markdown
Collaborator Author

@grantfirl I think it would make sense for you to take a close look, because I haven't done any testing of other cases than twpice (in particular, I have not updated any files that relate to prescribed surface fluxes). Note that this PR can go in later than all the others, the functionality of NEMSfv3gfs doesn't depend on it.

Copy link
Copy Markdown
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.

The changes look fine to me, although I did not test. I'll approve these so that we can merge this update in and I'll do some testing/troubleshooting after the fact.

@grantfirl grantfirl merged commit a320b09 into NCAR:master Jun 27, 2018
dustinswales pushed a commit to dustinswales/ccpp-scm that referenced this pull request May 16, 2022
…from-nemsfv3gfs

RRTMG lw/sw radiation updates from NEMSfv3gfs
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.

2 participants