Skip to content

[FEATURE] add trace heatmap chart plugin - #724

Open
shahrokni wants to merge 1 commit into
mainfrom
feat/tracheheatmapchart2
Open

[FEATURE] add trace heatmap chart plugin#724
shahrokni wants to merge 1 commit into
mainfrom
feat/tracheheatmapchart2

Conversation

@shahrokni

@shahrokni shahrokni commented Jul 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Relates to perses/perses#3942

This is the bare minimum trace heat map chart. All it needs is traces and the exponential base (default 2) to bucketize the traces according to their time and duration.

The x-axis and y-axis represent the time and duration buckets respectively. Both buckets are generated dynamically.
The following picture shows a base 2 bucketing.
It also supports base 10. Choosing a proper base may vary by different cases.

Demo

image

Same queries and traces, different bases

image

Next Steps?

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

@shahrokni
shahrokni force-pushed the feat/tracheheatmapchart2 branch 3 times, most recently from c40e966 to 139f2ea Compare July 16, 2026 12:35
@shahrokni
shahrokni force-pushed the feat/tracheheatmapchart2 branch 5 times, most recently from b33097f to cad2d04 Compare July 21, 2026 12:18
shahrokni
shahrokni commented Jul 21, 2026
View reviewed changes
Comment thread traceheatmapchart/schemas/traceheatmapchart.json Outdated
shahrokni
shahrokni commented Jul 21, 2026
View reviewed changes
Comment thread traceheatmapchart/src/traceheatmap-chart-model.ts Outdated
shahrokni
shahrokni commented Jul 21, 2026
View reviewed changes
Comment thread traceheatmapchart/src/traceheatmap-time-bucket-util.ts
shahrokni
shahrokni commented Jul 21, 2026
View reviewed changes
Comment thread traceheatmapchart/src/traceheatmap-time-bucket-util.ts
@shahrokni
shahrokni force-pushed the feat/tracheheatmapchart2 branch 3 times, most recently from e90da96 to 30c30f1 Compare July 21, 2026 21:57
@shahrokni
shahrokni force-pushed the feat/tracheheatmapchart2 branch from 30c30f1 to 69d4258 Compare July 21, 2026 22:01
@shahrokni

shahrokni commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

FINAL TEST IN PROGRESS ...

IMPORTANT: By changing the query type the rest of query spec lags and sometimes does not even load! This not coming from my change. Sounds like an old issue. Checking what could be the issue...

@shahrokni
shahrokni marked this pull request as ready for review July 21, 2026 22:07
@shahrokni
shahrokni requested review from a team, AntoineThebaud and Nexucis as code owners July 21, 2026 22:07
@shahrokni
[FEATURE] add trace heatmap chart plugin
ec0a324 
Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Mahmoud Shahrokni <seyedmahmoud.shahrokni@amadeus.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>

Signed-off-by: Seyed Mahmoud Shahrokni <39291137+shahrokni@users.noreply.github.com>
@shahrokni
shahrokni force-pushed the feat/tracheheatmapchart2 branch from 69d4258 to ec0a324 Compare July 24, 2026 09:33
jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread scripts/npm/npm.go

func MustGetWorkspaces(dirPath string) []string {
excludedWorkspaces := []string{"e2e"}
excludedWorkspaces := []string{"e2e", "traceheatmapchart"}

@jgbernalp jgbernalp Jul 24, 2026

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.

?

@shahrokni shahrokni Jul 27, 2026 •
edited
Loading

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.

At the moment this plugin does not include any Go files. Therefore, the pipeline throws an exception. For now, we can exclude the plugin from this part of the pipeline.

@jgbernalp jgbernalp Jul 27, 2026

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.

not sure why the plugin should not include go files. At least the sdk related files should exist.

jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread traceheatmapchart/.cjs.swcrc

@jgbernalp jgbernalp Jul 24, 2026

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.

we don't need these files as they are included in the repo root.

jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread traceheatmapchart/.swcrc

@jgbernalp jgbernalp Jul 24, 2026

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.

we don't need these files as they are included in the repo root.

jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread traceheatmapchart/.gitignore

@jgbernalp jgbernalp Jul 24, 2026

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.

we don't need these files as they are included in the repo root.

jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread traceheatmapchart/.eslintrc.js

@jgbernalp jgbernalp Jul 24, 2026

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.

we don't need these files as they are included in the repo root.

jgbernalp
jgbernalp reviewed Jul 24, 2026
View reviewed changes
Comment thread traceheatmapchart/LICENSE

@jgbernalp jgbernalp Jul 24, 2026

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.

we don't need these files as they are included in the repo root.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jgbernalp jgbernalp jgbernalp left review comments
@andreasgerstmayr andreasgerstmayr Awaiting requested review from andreasgerstmayr
@Nexucis Nexucis Awaiting requested review from Nexucis Nexucis is a code owner
@AntoineThebaud AntoineThebaud Awaiting requested review from AntoineThebaud AntoineThebaud is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shahrokni @jgbernalp