-
Notifications
You must be signed in to change notification settings - Fork 358
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
Skip building arrays for surface absorption when surface fluxes are disabled #6065
Conversation
@jonbob and @mark-petersen No rush on this. It's just a small change that is convenient for certain standalone cases. |
@mark-petersen please review when you can. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. Thanks @cbegeman for your code improvement and attention to detail. I agree that the forcing arrays do not need to be built when surfaces fluxes are off. I also checked that the initialized arrays are not used anywhere else when either config_disable_thick_sflux
or config_disable_tr_sflux
are true.
@mark-petersen Thank you for reviewing! |
Skip building arrays for surface absorption when surface fluxes are disabled When both thickness and tracer surface fluxes are disabled with the config_disable_* flags, skip building the transmission coefficient arrays and related arrays. [BFB] mpaso standalone only
passes:
merged to next |
@jonbob Thank you for your testing! |
merged to master |
When both thickness and tracer surface fluxes are disabled with the
config_disable_*
flags, skip building the transmission coefficient arrays and related arrays.