search

ocurrent / current-bench

Experimental benchmarking infrastructure using OCurrent pipelines
Apache License 2.0
33 stars 17 forks source link

Align the graphs vertically #315

Closed art-w closed 2 years ago

art-w commented 2 years ago

The UI shows the graphs in multiple columns. This makes it hard to read the regressions, because you have to visually jump horizontally and attempt to align the metric variations. See also https://github.com/ocurrent/current-bench/issues/247

It would probably work better if the graphs were full width, such that the vertical alignment is always correct. The height of the graphs could be smaller to not loose on information density.

See https://ziglang.org/perf/ for an example, where the x-axis is also using time and not just evenly spread commits (otherwise a big regression followed by a quick fix can be given too much importance in the timeline!)

punchagan commented 2 years ago

Should we close this and open a separate issue or edit this issue to reflect that we probably want to think about using time as x-axis instead of evenly spaced commits?

art-w commented 2 years ago

Sure, the time as x-axis is only a suggestion! But it comes with its own challenges with the graph library ^^

punchagan commented 2 years ago

Closed in #320