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

Add optional scaling to RRTMGP flux adjustment #668

Merged
merged 14 commits into from
Jul 9, 2021

Conversation

dustinswales
Copy link
Collaborator

This PR contains improvements to how the LW heating-rate profile is updated between calls to the RRTMGP radiation scheme.
The adjusted net LW flux is computed following Hogan and Bozzo 2015 (10.1002/2015MS000455)
Optionally, the flux adjustment can be damped with height using a logistic function. This was found to be necessary when using configurations with very high/thin layers (e.g. L127). The shape of the damping function can be controlled by namelist parameters.

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Looks pretty straightforward, just one question for my understanding.

@@ -267,9 +276,11 @@ subroutine dcyc2t3_run &

! Vertical ordering?
if (p_lev(1,1) .lt. p_lev(1, levs)) then
iSFC = levs
iSFC = levs + 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Was that an actual bug? Is iSFC for interfaces (which is always levs+1)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes^2.

@climbfuji
Copy link
Collaborator

This PR has been pulled into #680 and will be merged as part of it.

grantfirl added a commit that referenced this pull request Jul 9, 2021
Add extended diagnostic output from Thompson MP, includes "Add optional scaling to RRTMGP flux adjustment" (#668), fix index bug in m_micro.F90
@grantfirl grantfirl merged commit 3542434 into NCAR:main Jul 9, 2021
@dustinswales dustinswales deleted the hotfix_AddPLimitToLWJacAdj branch September 17, 2021 17:34
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.

3 participants