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

Simple fix to prevent negative tauaer in RRTMG LW #521

Merged

Conversation

dr0cloud
Copy link
Collaborator

RRTMG LW does not scatter and so it must extract the absorption aerosol optical depth. This should be taua_ext - taua_scat, which in the parlance of the un-normalized radiation variables in the RRTMG LW section is TAUA - SSAA.
Unfortunately this difference is sometimes a small negative value. This is not the fault of the Radiation GC, since it gets these two variables directly by callback to the aerosol system. It should really be fixed on the aerosol side, but as a simple mitigation for now, we make the fix max(TAUA - SSAA, 0.).

@dr0cloud dr0cloud added the bugfix This fixes a bug label Jan 19, 2022
@dr0cloud dr0cloud requested a review from a team as a code owner January 19, 2022 22:50
@dr0cloud dr0cloud self-assigned this Jan 19, 2022
@dr0cloud dr0cloud added bugfix This fixes a bug Non 0-diff The changes in this pull request are non-zero-diff 0 diff The changes in this pull request have verified to be zero-diff with the target branch. and removed bugfix This fixes a bug Non 0-diff The changes in this pull request are non-zero-diff labels Jan 20, 2022
@dr0cloud
Copy link
Collaborator Author

Its technically zero-diff label because any negative tauaer would have caused a crash so not run at all!

@sdrabenh sdrabenh merged commit 67303bd into develop Feb 3, 2022
@sdrabenh sdrabenh deleted the feature/pnorris/#520-negative-tauaer-ocurring-in-RRTMG-LW branch February 3, 2022 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. bugfix This fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants