ocaml-bench / sandmark

A benchmark suite for the OCaml compiler
The Unlicense
81 stars 38 forks source link

Fix normalise error when different number of benchmarks in baseline #351

Closed punchagan closed 2 years ago

punchagan commented 2 years ago

When displaying plots for the sequential benchmarks, we normalise the metrics based on the baseline values. The code in normalise fails with an error when the list of benchmarks in the baseline differ from the benchmarks in the variants that are being compared against the baseline. This commit cleans up the code to use pd.DataFrame.pivot and pd.DataFrame.melt to fix this error.

shakthimaan commented 2 years ago

LGTM.