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 update brings a change in answers to one test on Derecho #2729

Closed
ekluzek opened this issue Aug 24, 2024 · 2 comments
Closed

PPE update brings a change in answers to one test on Derecho #2729

ekluzek opened this issue Aug 24, 2024 · 2 comments
Assignees
Labels
bug something is working incorrectly closed: duplicate Issue already reported, or changes included in a different PR

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Aug 24, 2024

Brief summary of bug

#2689 brought in a change in answers to the ctsm5.2.023 to a single test:

ERP_P64x2_Lm13.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings

General bug information

CTSM version you are using: ctsm5.2.023-7-g3d0aeaedf

Does this bug cause significantly incorrect results in the model's science? ?

Configurations affected: Matrix on for a transient case

Details of bug

clm0 history fields that change are:

 RMS LIT_METN_Cap_vr                  1.7279E-09            NORMALIZED  2.6925E-09
 RMS LIT_METC_Cap_vr                  1.1491E-03            NORMALIZED  4.0341E-05
@ekluzek ekluzek added the bug something is working incorrectly label Aug 24, 2024
@ekluzek ekluzek added this to the cesm3_0_beta03 milestone Aug 24, 2024
@slevis-lmwg
Copy link
Contributor

@ekluzek I had documented this as "expected" in my PPE PR, but I may have misspoken. This is the lineage of the problem as I understand it:

So, in the end, are these (in #2689, #2588, and #2545) three problems, two problems, or one problem? I treated them as one "expected" problem.

@ekluzek ekluzek added the closed: duplicate Issue already reported, or changes included in a different PR label Aug 26, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Aug 26, 2024

@slevis-lmwg yep you are right this is a duplicate of #2619. So not really a problem.

I did end up in the rabbit hole of figuring out what might have changed in the PPE PR. I saw that c3psn was different, but showed that it didn't change this case. So this ended up largely being a waste of time, but it did validate the PPE PR.

@ekluzek ekluzek closed this as completed Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly closed: duplicate Issue already reported, or changes included in a different PR
Projects
Status: Done
Development

No branches or pull requests

2 participants