Skip to content

Commit db8c8d3

Browse files
committed
more succinct benchmark summaries
Signed-off-by: Will Manning <will@willmanning.io>
1 parent da4c196 commit db8c8d3

1 file changed

Lines changed: 25 additions & 41 deletions

File tree

scripts/compare-benchmark-jsons.py

Lines changed: 25 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -536,61 +536,45 @@ def main() -> None:
536536
else format_performance(geo_mean_ratio, improvement_threshold, regression_threshold, "overall")
537537
)
538538

539-
summary_lines = [
540-
"## Summary",
541-
"",
542-
f"- **Overall**: {overall_performance}",
543-
]
539+
statistical_analysis = build_statistical_analysis(df3, threshold_pct)
540+
verdict = build_verdict(statistical_analysis) if statistical_analysis is not None else None
541+
542+
heading = f"## Benchmarks: {benchmark_name}" if benchmark_name else "## Benchmarks"
543+
summary_lines = [heading, ""]
544+
545+
if verdict is not None:
546+
summary_lines.append(f"**Summary**: {verdict['status']}<br>")
547+
summary_lines.append(f"**Confidence**: {verdict['confidence']}<br>")
548+
summary_lines.append(f"**Attributed Vortex impact**: {verdict['impact']}<br>")
549+
summary_lines.append(f"**Environment shift**: {verdict['environment_shift']}<br>")
550+
551+
if statistical_analysis is not None:
552+
residual_noise = format_ratio_change(statistical_analysis["residual_noise_ratio"])
553+
summary_lines.append(f"**Residual noise**: {residual_noise}<br>")
554+
555+
polish = statistical_analysis["median_polish"]
556+
if polish is not None:
557+
summary_lines.append(
558+
f"**Median polish overall**: {format_ratio_change(float(np.exp(polish.overall)))}<br>"
559+
)
560+
561+
summary_lines.append(f"**Overall**: {overall_performance}<br>")
544562
if len(vortex_df) > 0:
545563
vortex_performance = format_performance(
546564
vortex_geo_mean_ratio,
547565
improvement_threshold,
548566
regression_threshold,
549567
"vortex",
550568
)
551-
summary_lines.append(f"- **Vortex**: {vortex_performance}")
569+
summary_lines.append(f"**Vortex**: {vortex_performance}<br>")
552570
if len(parquet_df) > 0:
553571
parquet_performance = format_performance(
554572
parquet_geo_mean_ratio,
555573
improvement_threshold,
556574
regression_threshold,
557575
"parquet",
558576
)
559-
summary_lines.append(f"- **Parquet**: {parquet_performance}")
560-
561-
statistical_analysis = build_statistical_analysis(df3, threshold_pct)
562-
verdict = build_verdict(statistical_analysis) if statistical_analysis is not None else None
563-
if verdict is not None:
564-
summary_lines.extend(
565-
[
566-
"",
567-
"## Verdict",
568-
"",
569-
f"**{verdict['status']}**",
570-
f"- **Attributed Vortex impact**: {verdict['impact']}",
571-
f"- **Confidence**: {verdict['confidence']}",
572-
f"- **Environment shift**: {verdict['environment_shift']}",
573-
]
574-
)
575-
576-
if statistical_analysis is not None:
577-
systemic_shift = format_ratio_change(statistical_analysis["systemic_shift_ratio"])
578-
control_sigma = format_ratio_change(float(np.exp(statistical_analysis["systemic_shift_std"])))
579-
residual_noise = format_ratio_change(statistical_analysis["residual_noise_ratio"])
580-
summary_lines.extend(
581-
[
582-
"",
583-
"## Statistical Summary",
584-
"",
585-
f"- **Systemic shift ({CONTROL_FORMAT} controls)**: {systemic_shift}",
586-
f"- **Control sigma**: {control_sigma}",
587-
f"- **Residual noise**: {residual_noise}",
588-
]
589-
)
590-
591-
polish = statistical_analysis["median_polish"]
592-
if polish is not None:
593-
summary_lines.append(f"- **Median polish overall**: {format_ratio_change(float(np.exp(polish.overall)))}")
577+
summary_lines.append(f"**Parquet**: {parquet_performance}")
594578

595579
print("\n".join(summary_lines))
596580
print("")

0 commit comments

Comments
 (0)