The GitHub Actions job "Tests (ARM)" on airflow.git/v3-3-test has failed.
Run started by GitHub user bbovenzi (triggered by bbovenzi).

Head commit for run:
af1e5b8904e8c1c3d01f4276561c147ee3c594d3 / github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
[v3-3-test] UI: Make duration charts readable at a glance (#70142) (#70197)

* UI: Make duration charts readable at a glance

The Airflow 3 duration chart dropped several things the Airflow 2 version
did well, and what is left is hard to read at the size it is given.

Durations on the value axis are the main cost: HH:mm:ss has to be decoded
tick by tick before the magnitude is apparent, and Chart.js picks decimal
steps, so a busy Dag lands on ticks like 26m 40s. The two reference lines
report the mean, which one stuck run drags well above where runs actually
sit, and both labels are pinned to the same edge so they overlap each other
and the most recent bars. Nothing names the two series. Finally the card is
pinned to 350px on a page with room to spare, which is what makes the bars
too small to compare in the first place.

The bars were also stacked on the index axis only, so queued time was drawn
behind run time rather than underneath it, and was effectively invisible.

* Add newsfragment for the duration chart changes

* UI: Pin the duration chart height

Letting the card flex horizontally handed its height to Chart.js' default 2:1
aspect ratio, which the old fixed 350px width had been capping as a side
effect. On a 1400px row the chart came out 496px tall, and on a 2560px monitor
it would have passed 1000px and swallowed the page it sits on.

* UI: Apply duration chart review feedback

The two overview pages had drifted apart for no reason: one card flexed, the
other sat in a single-column grid with only a max width, so the same chart
answered to two different sets of rules. They now share one Box, which also
gives the Task overview the cap the Dag overview needed to stop the chart
spanning an ultrawide monitor.

The reference line reads "Total:" rather than "Median total:", and the
newsfragment is dropped as not warranted for this change.
(cherry picked from commit ae756a97b3c130ea90555421c9cd0de838499f9e)

Co-authored-by: Eddie Roman <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/29862204171

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to