Skip to content

Perf (and AB-perf) consolidation#8000

Draft
maxtropets wants to merge 3 commits into
microsoft:mainfrom
maxtropets:f/ab-bench-plots
Draft

Perf (and AB-perf) consolidation#8000
maxtropets wants to merge 3 commits into
microsoft:mainfrom
maxtropets:f/ab-bench-plots

Conversation

@maxtropets

@maxtropets maxtropets commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator
  • Refactor AB-bench to be up-to-date with main
  • Change main + AB style
    • Plots bigger
    • Left->right instead of top->down

@maxtropets maxtropets self-assigned this Jun 29, 2026
@maxtropets maxtropets force-pushed the f/ab-bench-plots branch 2 times, most recently from d5ec504 to a75e4d0 Compare June 30, 2026 15:49
@achamayou

Copy link
Copy Markdown
Member

The text on the charts is now very small, and cannot be read at all if they are not expanded. The single column layout only uses a very small amount of horizontal space.
With the table of links gone, clicking through to the commit that caused the regression is no longer possible. It would be nicer to do on the plot directly of course, but I did not find a way to do that with mermaid.

Comment thread scripts/perf_report.py
@@ -0,0 +1,364 @@
# Copyright (c) Microsoft Corporation. All rights reserved.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Context: this's almost all moved from perf_summary for reuse by AB, except swapping plots axis

@maxtropets maxtropets changed the title [Draft] Perf (and AB-perf) consolidation Perf (and AB-perf) consolidation Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants