From 17d1d2ab67a0e3b2379e257658cac32bd943f6de Mon Sep 17 00:00:00 2001 From: Alex Jones Date: Mon, 25 May 2026 00:19:44 +0100 Subject: [PATCH] fix: status shown for combined instrumentation jobs An accidental omission of the intended `omit_status=True` keyword argument meant that for visualizations of the longest tests, where many jobs are combined into a single bar (for performance reasons), the status of the shortest such job was always being reported, instead of not reporting any status like was intended (since there could be variation amongst the combined jobs). Signed-off-by: Alex Jones --- src/dvsim/instrumentation/report/longest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/dvsim/instrumentation/report/longest.py b/src/dvsim/instrumentation/report/longest.py index 926fc7e7..05fcc59c 100644 --- a/src/dvsim/instrumentation/report/longest.py +++ b/src/dvsim/instrumentation/report/longest.py @@ -261,7 +261,7 @@ def _make_combined_bar_info( f"Mean duration: {format_time_metric(avg_duration)}", f"Minimum duration: {format_time_metric(min_duration)}", ] - hover = make_job_metadata_hover(test.name, extra_timing_info, meta) + hover = make_job_metadata_hover(test.name, extra_timing_info, meta, omit_status=True) return total_duration, hover