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

PPE change to sa_leaf in CanopyFluxesMod.F90 #2788

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

olyson
Copy link
Contributor

@olyson olyson commented Sep 25, 2024

Description of changes

Add change to sa_leaf that was in PPE branch but is not on master

Specific notes

Contributors other than yourself, if any: @olyson

CTSM Issues Fixed (include github issue #): #2777

Are answers expected to change (and if so in what way)? Yes. See #2777 for results.

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? Yes. No.

Testing performed, if any:
No test suite testing thus far.

@olyson olyson added bug something is working incorrectly PR status: awaiting review Work on this PR is paused while waiting for review. science Enhancement to or bug impacting science non-bfb Changes answers (incl. adding tests) labels Sep 25, 2024
@olyson olyson self-assigned this Sep 25, 2024
Copy link
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

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

@olyson found this line missing from the recent PPE code that came to master. That recent code came in as b4b, while the modification here changes answers. Keith already evaluated the effect of this modification in #2777.

To me it seems a good idea to merge this quickly.

@slevis-lmwg slevis-lmwg removed the PR status: awaiting review Work on this PR is paused while waiting for review. label Sep 26, 2024
@samsrabin samsrabin removed their assignment Oct 3, 2024
@samsrabin
Copy link
Collaborator

@wwieder Removing my assignment, as with the change to our queue order I no longer care when this comes in ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly non-bfb Changes answers (incl. adding tests) science Enhancement to or bug impacting science
Projects
Status: In progress - master/b4b-dev
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants