Skip to content

Conversation

@forsyth2
Copy link
Collaborator

@forsyth2 forsyth2 commented Dec 8, 2025

Summary

Objectives:

  • Automate process to go from high-level overview of testing diffs to high-resolution singular images. That is, provide a simple link.

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 Dec 8, 2025
@forsyth2 forsyth2 added Testing Files in `tests` modified semver: small improvement Small improvement (will increment patch version) labels Dec 8, 2025
Copy link
Collaborator Author

@forsyth2 forsyth2 left a comment

Choose a reason for hiding this comment

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

Will test this PR on the next few zppy test runs, e.g., for E3SM-Project/e3sm_diags#1021 & #760

pdf_url = web_portal_base_url + "/" + web_subdir + "/" + pdf_name
diff_url = web_portal_base_url + "/" + web_subdir + short_title + "_diff.png"
Example:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@forsyth2
Copy link
Collaborator Author

forsyth2 commented Dec 17, 2025

I have used this commit to test both E3SM-Project/e3sm_diags#1021, the 2025-12-16 weekly test, #765. It works as intended; merging.

@forsyth2 forsyth2 merged commit c4d9db1 into main Dec 17, 2025
7 checks passed
@forsyth2 forsyth2 deleted the issue-766-test-links branch December 17, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver: small improvement Small improvement (will increment patch version) Testing Files in `tests` modified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Testing -- Add links to titles of each row of the image diff grid

2 participants