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.

Sign up for GitHub

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

Jump to bottom

Revisions to repair iovr=5 cloud overlap option #829

Closed
wants to merge 3 commits into from
Closed

Revisions to repair iovr=5 cloud overlap option #829

wants to merge 3 commits into from

Conversation

mjiacono
Copy link
Collaborator

This PR addresses part 2 of issue #748 to activate the exponential-random cloud overlap method (iovr=5) in RRTMG.

@grantfirl
Copy link
Collaborator

@mjiacono Would you please merge in the latest ccpp-physics main branch to your PR branch and push? It looks like there are some code changes in radiation_clouds.F that Greg Thompson recently introduced via #809 (that was merged recently) that should probably not be reverted back.

@mjiacono mjiacono closed this Jan 20, 2022
@grantfirl grantfirl mentioned this pull request Jan 20, 2022
Merged
@grantfirl
Copy link
Collaborator

Replaced by #830.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@climbfuji climbfuji Awaiting requested review from climbfuji

@grantfirl grantfirl Awaiting requested review from grantfirl grantfirl is a code owner

@mkavulich mkavulich Awaiting requested review from mkavulich

@SamuelTrahanNOAA SamuelTrahanNOAA Awaiting requested review from SamuelTrahanNOAA

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mjiacono @grantfirl