Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

CI: use live base tip for mpl baseline generation#580

Merged
cvanelteren merged 2 commits into
mainUltraplot/UltraPlot:mainfrom
fix/ci-baseline-base-tipUltraplot/UltraPlot:fix/ci-baseline-base-tipCopy head branch name to clipboard
Feb 11, 2026
Merged

CI: use live base tip for mpl baseline generation#580
cvanelteren merged 2 commits into
mainUltraplot/UltraPlot:mainfrom
fix/ci-baseline-base-tipUltraplot/UltraPlot:fix/ci-baseline-base-tipCopy head branch name to clipboard

Conversation

@cvanelteren

Copy link
Copy Markdown
Collaborator

This fixes recurring image-compare failures (notably test_centered_legends) across PRs where the compare job was generating and restoring baselines keyed by github.event.pull_request.base.sha, which can be stale relative to the current base branch tip.

Changes:

  • Resolve base ref and sha at runtime from origin pull_request.base.ref.
  • Use the resolved sha for cache key and baseline checkout.
  • Bump baseline cache namespace from v3 to v4 to invalidate stale entries.

This keeps baseline generation aligned with the actual current base branch and avoids persistent false diffs caused by old baseline snapshots.

Fix repeated centered-legend image mismatches across PRs caused by comparing against stale baseline commits from pull_request payload SHA.

Workflow changes: resolve base ref/sha at runtime from origin/<base.ref>, use that SHA for cache key and baseline checkout, and bump baseline cache namespace to v4 to invalidate stale cache entries.
@cvanelteren cvanelteren merged commit 84669b6 into main Feb 11, 2026
7 of 8 checks passed
@cvanelteren cvanelteren deleted the fix/ci-baseline-base-tip branch February 11, 2026 09:14
K-Mirembe-Mercy pushed a commit to K-Mirembe-Mercy/UltraPlot that referenced this pull request Mar 22, 2026
Fix repeated centered-legend image mismatches across PRs caused by comparing against stale baseline commits from pull_request payload SHA.

Workflow changes: resolve base ref/sha at runtime from origin/<base.ref>, use that SHA for cache key and baseline checkout, and bump baseline cache namespace to v4 to invalidate stale cache entries.
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.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.