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

Strange diff for SCREAMv0 DP test #6145

Open
rljacob opened this issue Jan 10, 2024 · 0 comments
Open

Strange diff for SCREAMv0 DP test #6145

rljacob opened this issue Jan 10, 2024 · 0 comments
Assignees
Labels
Atmosphere Testing Anything related to unit/system tests

Comments

@rljacob
Copy link
Member

rljacob commented Jan 10, 2024

After PR #6113 was merged, the SMS_Ln9_P24x1.ne4_ne4.FDPSCREAM-ARM97 test showed a few diffs from baselines in the eam.h0 file.

SUMMARY of cprnc:
 A total number of    243 fields were compared
          of which     11 had non-zero differences

The variables were QDIFF, TDIFF, and *_RES variables.

The PR gave the case 2 new namelist variables and values:
pgrad_correction = 1
hv_ref_profiles = 6

Logically, this change should be BFB for this case.

@bogensch noted:

that is odd as all those variables (QDIFF, TDIFF, *RES) are computed from the state. So it's weird that one set would be b4b and the other not.... One commonality of the output variables listed that are not b4b is that they are all computed in the components/eam/src/dynamics/se/se_iop_intr_mod.F90 file. This provides a clue but looking at the PR (and the namelist changes ....), I can't explain off of the top of my head why these diffs are present. I will have to chew on this.

@rljacob rljacob added Atmosphere Testing Anything related to unit/system tests labels Jan 10, 2024
@rljacob rljacob changed the title Strange diff for SCREAM test Strange diff for SCREAMv0 DP test Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Atmosphere Testing Anything related to unit/system tests
Projects
None yet
Development

No branches or pull requests

2 participants