Skip to content

Commit 9a93e42

Browse files
edmundmillerclaude
andcommitted
fix(report): dynamic height + force all labels on cost boxplot
With many processes (44+), the fixed 500px height caused ECharts to hide Y-axis labels. RSEQC_TIN's $3-9 box appeared to belong to RUSTQC. - Scale chart height to 25px per process (min 500px) - Set interval: 0 to force all Y-axis labels visible Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 7fcde02 commit 9a93e42

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

bin/benchmark_report_template.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -697,7 +697,6 @@ <h2 id="task-metrics"><svg class="h-icon sm"><use href="#ic-scatter"/></svg> Tas
697697

698698
const costBoxEl = document.createElement('div');
699699
costBoxEl.className = 'chart';
700-
costBoxEl.style.height = '500px';
701700
container.appendChild(costBoxEl);
702701

703702
setTimeout(() => {
@@ -718,12 +717,13 @@ <h2 id="task-metrics"><svg class="h-icon sm"><use href="#ic-scatter"/></svg> Tas
718717
}
719718
});
720719

720+
costBoxEl.style.height = Math.max(500, processes.length * 25 + 60) + 'px';
721721
echarts.init(costBoxEl, 'seqera').setOption({
722722
title: { text: 'Task cost ($) per process' },
723723
tooltip: { trigger: 'item' },
724724
grid: { top: 40, bottom: 20, containLabel: true },
725725
yAxis: { type: 'category', data: processes,
726-
axisLabel: { width: 220, overflow: 'truncate' },
726+
axisLabel: { width: 220, overflow: 'truncate', interval: 0 },
727727
inverse: true },
728728
xAxis: { type: 'value', name: 'Cost ($)', nameLocation: 'center', nameGap: 25 },
729729
series: [{ type: 'boxplot', data: boxData }],

0 commit comments

Comments
 (0)