Skip to content

Conversation

@forsyth2
Copy link
Collaborator

@forsyth2 forsyth2 commented Oct 24, 2025

Summary

Objectives:

  • Ensure tests pass on E3SM Unified 1.12.0rc3

Issue resolution:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Oct 24, 2025
@forsyth2 forsyth2 added semver: bug Bug fix (will increment patch version) Testing Files in `tests` modified priority: high High priority task (for next release) labels Oct 24, 2025
@forsyth2
Copy link
Collaborator Author

forsyth2 commented Oct 24, 2025

Remaining TODO for rc3:

  • Debug the Perlmutter & Compy errors
  • Rerun jobs on Perlmutter & Compy (if only tests are changed, reuse the Unified rc2 environment; otherwise, use a dev environment)
  • Determine what expected results to test against. Because the zppy tests aren't run frequently on Perlmutter & Compy, they are going to be out of date. Options:
    • Examine the diffs, checking if it's likely all of them are due to changes over this development period. If so, update the local machine's expected results.
    • Copy/transfer the Chrysalis expected results to Perlmutter & Compy and test against those. This is probably the more robust approach, unless the plotting packages produce results that differ greatly depending between machines.

@forsyth2
Copy link
Collaborator Author

The second commit, 0c97da8, undoes the NCO call change of #717. This needed to be undone because the NCO call was hard-coded to Chrysalis, thus causing the issue noted at #746 (comment).

Note that #734 should make this more painless, but I'm wary of adding a new parameter at this point in Unified testing. That should be an early PR in the next development period though.

@forsyth2
Copy link
Collaborator Author

forsyth2 commented Oct 27, 2025

Copy/transfer the Chrysalis expected results to Perlmutter & Compy and test against those. This is probably the more robust approach, unless the plotting packages produce results that differ greatly depending between machines.

I think this would actually be infeasible.

# cd into the expected results directory on Chrysalis
cd /lcrc/group/e3sm/public_html/zppy_test_resources

# Let's count how many times `lcrc` appears
grep -R -l "lcrc" * | wc -l
# 9465

# And how many times a `ac.<username>` path appears
grep -R -l "ac\." * | wc -l
# 934

In theory, we could systematically change the file paths per machine, but that's outside the scope of the Unified testing period. Therefore, we'll continue with what we've been doing, i.e., option 1:

Examine the diffs, checking if it's likely all of them are due to changes over this development period. If so, update the local machine's expected results.

@forsyth2 forsyth2 marked this pull request as ready for review October 28, 2025 21:06
@forsyth2 forsyth2 merged commit 17ce354 into main Oct 28, 2025
6 checks passed
@forsyth2 forsyth2 deleted the fixes-for-rc3 branch October 28, 2025 21:07
This was referenced Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high High priority task (for next release) semver: bug Bug fix (will increment patch version) Testing Files in `tests` modified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants