Skip to content

show benchmark uncertainty and axis controls - #20

Merged
winitzki merged 3 commits into
mainfrom
sergeiwinitzki/fixes-in-display
Sep 11, 2026
Merged

winitzki merged 3 commits into
mainfrom
sergeiwinitzki/fixes-in-display

Conversation

@winitzki

@winitzki winitzki commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Some small LLM-generated fixes in benchmark display, motivated by my recent work on benchmarks:

  • plots show error bars when stdev is available
  • the Y axes start from 0 by default
  • fix a bug with formatting the commit hashes in rare cases (commit hash 12345678e9 will no longer be parsed as a scientific-format float)
  • the "time" plot goes first if it exists, then all others
  • one can now load up to 500 commits when looking at the master branch, and add up to 50 master-branch commits when looking at a PR branch

After these changes, plots will look like this (cabal run benchmark-display -- --sqlite perf-2026-07-28.sqlite3 --branch master --limit 28 on the macbook):

Screenshot 2026-09-10 at 19 27 41

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread src/Perf/Web/Plot.hs
"xaxis"
.= object
[ "title" .= ("" :: Text),
"type" .= ("category" :: Text),

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This should fix the formatting git commit hashes as text and not as auto-detected type.

@winitzki
winitzki force-pushed the sergeiwinitzki/fixes-in-display branch from f062342 to 7669725 Compare September 11, 2026 10:21

@chrisdone-artificial chrisdone-artificial left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Smashing!

@winitzki
winitzki merged commit 173382f into main Sep 11, 2026
1 check passed
@winitzki
winitzki deleted the sergeiwinitzki/fixes-in-display branch September 11, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants