Skip to content

README info to reopen PR - #81

Draft
maritsandstad wants to merge 3 commits into
noresmfrom
new-branch-to-track-old-78
Draft

README info to reopen PR#81
maritsandstad wants to merge 3 commits into
noresmfrom
new-branch-to-track-old-78

Conversation

@maritsandstad

Copy link
Copy Markdown
Collaborator

Description:

This is just a copy of what was in #78. Why it doesn't show up any of the changes that were reverted on main, I am not quite sure of. There might be more reverts needed to disentangle, that is also to be investigated at a later stage.

Collaborators:

@mvdebolskiy , @kjetilaas , @JessicaNeedham @rosiealice

Expectation of Answer Changes:

Description of generative AI usage (as necessary)

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

All checklist items must be checked to enable merging this pull request:

Contributor

  • The in-code documentation has been updated with descriptive comments
  • The documentation has been assessed to determine if updates are necessary
  • Describe use of generative AI (if necessary)

Integrator

  • FATES PASS/FAIL regression tests were run
  • Evaluation of test results for answer changes was performed and results provided
  • FATES-CLM6 Code Freeze: satellite phenology regression tests are b4b

If satellite phenology regressions are not b4b, please hold merge and notify the FATES development team.

Documentation

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

@maritsandstad

Copy link
Copy Markdown
Collaborator Author

@mvertens this is where I have the reverted commit to include the fixes that failed the coupled tests before the summer. I am currently rerunning prealpha_noresm on Olivia for this whole thing so we can see what the starting point was, then from there we can try to triage our way back to figure out which of the PRs that went in here might be causing issues and which ones don't. They were all passing aux_clm_noresm tests before getting merged in.

@maritsandstad

Copy link
Copy Markdown
Collaborator Author

@mvertens have run the prealpha for this on Olivia got one fail for which was an interconnect issue which ran when I reran it, but this test failed because the run ran out of time:

/cluster/work/projects/nn9560k/masan/noresm/ERI.ne30pg3_tn14.N1850.olivia_intel.20260804_134245_hvgieu.ref1/

Is that a thing that often happens on Olivia? Should I just try to rerun it?

@mvertens

mvertens commented Aug 5, 2026

Copy link
Copy Markdown

@maritsandstad - yes - this has happened to me quite often lately. If you resubmit it normally works.

@maritsandstad

Copy link
Copy Markdown
Collaborator Author

@mvertens , ok reran this and with that alll the prealpha tests on Olivia passed for this. Will run on Betzy tomorrow. Still trying to get files over for the aux_clm-tests the snail way. Should be faster if I can maybe just copy them from Betzy tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants