Skip to content

fix(bench): compact dashboard status block#79

Merged
polaz merged 1 commit into
mainfrom
fix/#79-bench-dashboard-status
Apr 7, 2026
Merged

fix(bench): compact dashboard status block#79
polaz merged 1 commit into
mainfrom
fix/#79-bench-dashboard-status

Conversation

@polaz
Copy link
Copy Markdown
Member

@polaz polaz commented Apr 7, 2026

Summary

  • replace oversized single-line Latest deltas by series status text with compact summary
  • show only top deviating series in a short list to avoid overflow noise
  • keep detailed data in chart/filters while making footer status readable

Validation

  • cargo nextest run --workspace
  • cargo build --workspace

Summary by CodeRabbit

  • Bug Fixes

    • Improved status messaging clarity and error handling in the benchmark dashboard.
    • Enhanced visual status indicators to distinguish between parity and out-of-band conditions.
  • New Features

    • Benchmark dashboard now displays a focused series list (up to 8 items) when performance deviations are detected.
    • Added abbreviated series label formatting for improved readability.

@polaz polaz merged commit a403575 into main Apr 7, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 7, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: b92738e6-0c63-4333-bb63-a9e6e0941b94

📥 Commits

Reviewing files that changed from the base of the PR and between b6f9a01 and 2aa75b3.

📒 Files selected for processing (1)
  • .github/bench-dashboard/index.html

📝 Walkthrough

Walkthrough

Modified the benchmark dashboard status display to add helper functions for DOM updates and restructure multi-series status rendering. Now displays per-series deviation metrics and conditionally lists "focus" series based on parity band status, replacing a single-line message.

Changes

Cohort / File(s) Summary
Benchmark Dashboard Status UI
.github/bench-dashboard/index.html
Added setStatusText() and compactSeriesLabel() helper functions. Refactored updateStatus() to compute per-series deviation, build structured status UI with summary and focus list (up to 8 series), and set status class based on parity band detection. Updated error handler and added CSS for status list styling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Whiskers twitched at the dashboard's glow,
Status updates now steal the show,
Focus lists sorted by deviation high,
Parity bands beneath the sky,
The bench board hops with info so neat!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#79-bench-dashboard-status

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

@polaz polaz deleted the fix/#79-bench-dashboard-status branch April 7, 2026 20:42
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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