Skip to content

Commit 15d1e74

Browse files
committed
web-app: make multi-latency plots taller and nicer looking, when plotting 50+ runs
1 parent 0eb9e5e commit 15d1e74

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

app/web-app/index.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -832,16 +832,17 @@ app.component('MultiLatencyLineChart', {
832832
style="
833833
position: relative;
834834
width: 800px;
835-
height: 500px;
836835
padding: 50px 20px;
837836
margin: auto;
838837
"
839838
>
840839
<h1>{{benchName.replace("-k6-custom","").replaceAll("_"," ")}}</h1>
841840
<hr />
842-
<canvas id="chart-container" ref="chartElem"></canvas>
841+
<canvas height="250" ref="chartElem"></canvas>
843842
</div>
844843
`,
844+
// ^^^ NOTE: before changing height styling above, make sure it looks good
845+
// with a chart of at least 50 runs
845846
setup(props) {
846847
const chartElem = ref(null)
847848
onMounted(() => {
@@ -852,8 +853,8 @@ app.component('MultiLatencyLineChart', {
852853
label,
853854
data,
854855
fill: false,
855-
borderWidth: 2,
856-
pointRadius: 2.5,
856+
borderWidth: 1.5,
857+
pointRadius: 0, // disable points, which looks cluttered
857858
// This is a heatmap-style red to blue color scheme which lets us show
858859
// the results "fading back in time":
859860
borderColor: redToBlue(ix, props.benchData.length)

0 commit comments

Comments
 (0)