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

cam6_3_145: New PUMAS External with adjusted vapor deposition onto snow #938

Merged
merged 7 commits into from
Jan 5, 2024

Conversation

Katetc
Copy link
Collaborator

@Katetc Katetc commented Dec 7, 2023

Issue #936 - A one-line change in PUMAS resulting in a new tag for the external. This needs to be merged after the ML PR to cam_development (PR #858 ).

resolves #936

Includes Cheryl's PR
closes #948
Closes #947

Run to confirm the impact of this change is at: NCAR/amwg_dev#445

@Katetc Katetc added the answer changing answer changing tag label Dec 7, 2023
@Katetc Katetc self-assigned this Dec 7, 2023
Copy link

@adamrher adamrher left a comment

Choose a reason for hiding this comment

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

Tough review, but I managed to get through it.

@cacraigucar
Copy link
Collaborator

Requested re-review from Adam and initial review from @PeterHjortLauritzen . This is not for reviewing the code, but accepting the science changes which are coming in.

Copy link

@adamrher adamrher left a comment

Choose a reason for hiding this comment

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

Approved. based on latest science results NCAR/amwg_dev#445

Copy link
Collaborator

@PeterHjortLauritzen PeterHjortLauritzen left a comment

Choose a reason for hiding this comment

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

Thanks @adamrher and @Katetc

@cacraigucar cacraigucar removed the request for review from nusbaume December 19, 2023 17:41
@cacraigucar cacraigucar changed the title New PUMAS External with adjusted vapor deposition onto snow cam6_3_145: New PUMAS External with adjusted vapor deposition onto snow Dec 19, 2023
@peverwhee peverwhee changed the title cam6_3_145: New PUMAS External with adjusted vapor deposition onto snow cam6_3_145?: New PUMAS External with adjusted vapor deposition onto snow Jan 2, 2024
@peverwhee peverwhee changed the title cam6_3_145?: New PUMAS External with adjusted vapor deposition onto snow cam6_3_145: New PUMAS External with adjusted vapor deposition onto snow Jan 2, 2024
@Katetc Katetc merged commit 452cb5b into ESCOMP:cam_development Jan 5, 2024
@Katetc Katetc deleted the katetc/rainevap_pr branch January 5, 2024 20:56
cacraigucar added a commit to cacraigucar/CAM that referenced this pull request Jan 19, 2024
Merge pull request ESCOMP#938 from PUMASDevelopment/katetc/rainevap_pr
gold2718 pushed a commit to gold2718/CAM that referenced this pull request May 2, 2024
Merge pull request ESCOMP#938 from PUMASDevelopment/katetc/rainevap_pr

cam6_3_145: New PUMAS External with adjusted vapor deposition onto snow
Issue ESCOMP#936 - A one-line change in PUMAS resulting in a new tag for the external. This needs to be merged after the ML PR to cam_development (PR ESCOMP#858 ).

resolves ESCOMP#936

Includes Cheryl's PR
closes ESCOMP#948
Closes ESCOMP#947

Run to confirm the impact of this change is at: NCAR/amwg_dev#445

ESCOMP commit: 452cb5b
adamrher added a commit to adamrher/CAM that referenced this pull request May 10, 2024
Merge pull request ESCOMP#938 from PUMASDevelopment/katetc/rainevap_pr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answer changing answer changing tag
Projects
Status: Tag
Development

Successfully merging this pull request may close these issues.

4 participants