Skip to content

feat(app): show percentile context in the heatmap hover tooltip - #2789

Merged
kodiakhq[bot] merged 2 commits into
ClickHouse:mainfrom
niladrix719:heatmap-toolpit-percentile#1911
Aug 4, 2026
Merged

kodiakhq[bot] merged 2 commits into
ClickHouse:mainfrom
niladrix719:heatmap-toolpit-percentile#1911

Conversation

@niladrix719

Copy link
Copy Markdown
Contributor

Summary

Hovering a heatmap cell will show percentile information in tooltip now

fixes #1911

Screenshots or video

Screenshot 2026-08-04 at 11 51 15 AM

How to test on Vercel preview

Preview routes: /search

Steps:

  1. Confirm a trace source (e.g. "otel_traces") is selected in the source selector; if not, select it.
  2. Click the "Event Deltas" tab in the left filter panel.
  3. Wait for the heatmap chart to render above the "Select an area on the chart above to enable comparisons" row.
  4. Hover the mouse over one of the colored heatmap cells.
  5. Verify the tooltip shows a "Y Value" line with a percentile in parentheses, e.g. "Y Value: 792ms (p99.1)".

Signed-off-by: Niladri Adhikary <niladrix719@gmail.com>
@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

@niladrix719 is attempting to deploy a commit to the HyperDX Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 188848f

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@greptile-apps

greptile-apps Bot commented Aug 4, 2026 •

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds cumulative percentile context to heatmap hover tooltips and unit coverage for aggregation, empty data, and bucket ordering.

  • Aggregates event counts across time buckets by Y bucket.
  • Displays the cumulative percentile alongside the hovered Y value.
  • Adds a patch changeset for @hyperdx/app.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
packages/app/src/components/DBHeatmapChart.tsx Adds cumulative per-bucket percentile computation and renders the result in the heatmap hover tooltip.
packages/app/src/components/tests/DBHeatmapChart.test.ts Covers pooled cumulative percentages, empty distributions, and numeric bucket ordering.
.changeset/heatmap-tooltip-percentile.md Records the tooltip enhancement as a patch release for the app package.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Data[Heatmap data] --> Aggregate[Aggregate counts by Y bucket]
  Aggregate --> Cumulative[Compute cumulative percentages]
  Hover[Hovered heatmap cell] --> Lookup[Look up Y-bucket percentile]
  Cumulative --> Lookup
  Lookup --> Tooltip[Render Y Value with percentile]
Loading

Reviews (2): Last reviewed commit: "Merge branch 'main' into heatmap-toolpit..." | Re-trigger Greptile

@pulpdrew pulpdrew 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.

LGTM, thanks for the contribution!

@kodiakhq
kodiakhq Bot merged commit 17408ef into ClickHouse:main Aug 4, 2026
23 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Event Deltas: Heatmap hover tooltip with percentile context

2 participants