Skip to content

fix: overhaul benchmark dashboard UI and fix broken URLs#374

Merged
strawgate merged 1 commit into
masterfrom
fix/dashboard-ui-overhaul
Mar 31, 2026
Merged

fix: overhaul benchmark dashboard UI and fix broken URLs#374
strawgate merged 1 commit into
masterfrom
fix/dashboard-ui-overhaul

Conversation

@strawgate
Copy link
Copy Markdown
Owner

Summary

Fixes multiple issues with the benchmark dashboard that made it non-functional and hard to use.

Bug Fixes

  • [object Object] errors: Run IDs were being used as objects instead of extracting the .id field, causing all data fetches to fail
  • Broken URLs: bench.yml workflow and benchmarking.md docs pointed to wrong domains (strawgate.comstrawgate.github.io) and paths (/dev/bench//bench/)
  • Undefined canvasId in run.html caused charts on the run detail page to silently fail

UX Improvements

  • Friendly scenario names: passthroughPassthrough (No Transforms), json_parseJSON Parsing & Extraction, etc.
  • Inline config snippets: Each scenario shows the actual SQL transform used (e.g. SELECT * FROM logs WHERE level_str IN ('WARN', 'ERROR')), so readers know exactly what the benchmark is doing
  • Removed stddev columns from competitive and run tables — throughput (LPS) is the metric people care about
  • Added navigation: "← Back to Docs" link on all dashboard pages connects back to the mdBook site

Styling

  • Replaced heavy glassmorphism (translucent blurs, radial gradients) with clean solid backgrounds
  • Improved text contrast for readability in both light and dark mode
  • Modernized typography with Inter font

Files Changed

File Change
bench/dashboard/style.css Full CSS overhaul
bench/dashboard/index.html Friendly names, nav link, ID extraction fix
bench/dashboard/competitive.html Friendly names, nav link, remove stddev, ID extraction fix
bench/dashboard/run.html Friendly names, nav link, remove stddev, canvasId fix
.github/workflows/bench.yml Fix dashboard URLs in issue reports
book/src/development/benchmarking.md Fix docs URL path

- Fix [object Object] errors by correctly extracting run IDs from index data
- Replace raw scenario names (passthrough, json_parse, filter) with
  friendly titles and inline SQL config snippets showing what each
  benchmark actually does
- Remove stddev columns from competitive and run detail tables
- Add 'Back to Docs' navigation links across all dashboard pages
- Modernize CSS: solid backgrounds, higher contrast, better typography
- Fix incorrect bench.yml URLs (strawgate.com → strawgate.github.io)
- Fix incorrect benchmarking.md URL path (/dev/bench/ → /bench/)
- Fix undefined canvasId bug in run detail page
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 31, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

This pull request updates benchmark dashboard infrastructure and presentation. It migrates GitHub Pages URLs from strawgate.com to strawgate.github.io in workflow configuration and documentation. Across three HTML dashboard files, it introduces consistent SCENARIO_NAMES mappings and helper functions to display human-readable scenario titles instead of raw IDs. The table layouts in competitive and run dashboards are modified to remove standard deviation columns. A comprehensive CSS refactor modernizes typography, introduces Inter webfont, adjusts color tokens and spacing, reworks navigation and card styling, and updates responsive breakpoints.

Possibly related PRs


Comment @coderabbitai help to get the list of available commands and usage tips.

@strawgate strawgate merged commit 6ab7864 into master Mar 31, 2026
8 of 11 checks passed
@strawgate strawgate deleted the fix/dashboard-ui-overhaul branch March 31, 2026 07:19
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.

1 participant