Skip to content

test: Regenerate matplotlib 3.11 plot references to match CI timezone#8706

Merged
aldbr merged 1 commit into
DIRACGrid:rel-v8r0from
chrisburr:fix/plot-tests-utc-v8r0
Jul 13, 2026
Merged

test: Regenerate matplotlib 3.11 plot references to match CI timezone#8706
aldbr merged 1 commit into
DIRACGrid:rel-v8r0from
chrisburr:fix/plot-tests-utc-v8r0

Conversation

@chrisburr

@chrisburr chrisburr commented Jul 12, 2026

Copy link
Copy Markdown
Member

The matplotlib 3.11 accounting-plot reference images added for the Python 3.14 DIRACOS build were rendered in a UTC environment. DIRAC's Graphs format time-axis labels in the machine's local timezone (GraphUtilities.add_time_to_title via time.localtime, and the *Graph classes via datetime.fromtimestamp) while labelling them "UTC", so the plots depend on the renderer's timezone.

Both the DIRAC and DIRACOS2 integration CIs render these plots in CET, so the UTC references were one hour off on the five time-axis plots (stackedline, cumulativeplot, timedstackedbarplot, qualityplot1, qualityplot2) and failed the comparison under matplotlib 3.11 in the DIRACOS2 (Python 3.14) build. The older <stem>.png references were already CET, which is why DIRAC's own CI (matplotlib 3.10) kept passing.

This regenerates those five references from the matplotlib 3.11 CI output so they match. No code change; the <stem>.png references are unchanged.

Note: the underlying local-time-labelled-"UTC" behaviour in Graphs is a pre-existing quirk, unrelated to this change.

The matplotlib 3.11 accounting-plot references (added for the Python 3.14
DIRACOS build) were rendered in a UTC environment, but DIRAC's Graphs format
time-axis labels in the machine's local timezone (GraphUtilities.add_time_to_title
via time.localtime, and the *Graph classes via datetime.fromtimestamp). The DIRAC
and DIRACOS2 integration CIs both render these plots in CET, so the UTC references
were one hour off on the five time-axis plots and failed under matplotlib 3.11.
Regenerate those five references from the matplotlib 3.11 CI output; the older
<stem>.png references are unchanged.
@chrisburr
chrisburr force-pushed the fix/plot-tests-utc-v8r0 branch from 6bd18ee to 16f5fcf Compare July 12, 2026 19:41
@chrisburr chrisburr changed the title test: Render accounting plot tests in UTC for reproducible time labels test: Regenerate matplotlib 3.11 plot references to match CI timezone Jul 12, 2026
@chrisburr
chrisburr marked this pull request as ready for review July 12, 2026 20:50
@aldbr
aldbr merged commit 462b6a3 into DIRACGrid:rel-v8r0 Jul 13, 2026
22 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants