Skip to content

feat: add pivot heatmaps and trends [CMP-48642] - #66

Merged
chaim0m merged 3 commits into
mainfrom
feat/chaim/pivot-visualization-CMP-48642
Aug 11, 2026
Merged

feat: add pivot heatmaps and trends [CMP-48642]#66
chaim0m merged 3 commits into
mainfrom
feat/chaim/pivot-visualization-CMP-48642

Conversation

@chaim0m

@chaim0m chaim0m commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add first-to-last trend values to pivot rows.
  • Add interactive pivot heatmaps, gated off for agents, pipes, and NO_COLOR.
  • Let --raw-numbers preserve epoch timestamps in table and TOON output.

This is split from the embedded dci-cli changes in doitintl/omni#60609. The body-validation compatibility bypass from the same upstream commit is already covered by #60 and is not duplicated here.

Stack

Depends on #64.

Validation

  • go test ./...
  • go test -race ./...
  • go vet ./...
  • go build ./...
  • golangci-lint run --new-from-rev=feat/chaim/report-presentation-CMP-48642 (0 issues)

No emulators were run.

Jira: CMP-48642

@chaim0m
chaim0m requested review from a team, apgiorgi, eranchetz and taltultc as code owners August 11, 2026 13:06
@chaim0m chaim0m added the review-effort: 2 Low review effort label Aug 11, 2026
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread pivot.go Outdated
Comment thread pivot.go
@chaim0m
chaim0m requested a review from yariv-doit August 11, 2026 13:41
@chaim0m
chaim0m force-pushed the feat/chaim/report-presentation-CMP-48642 branch from d929285 to 21b43e9 Compare August 11, 2026 13:48
Comment thread main.go Outdated

@yariv-doit yariv-doit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Rechecked at 1224c20. The previously raised blockers are fixed, the relevant tests pass, and the remaining inline note is non-blocking.

chaim0m and others added 2 commits August 11, 2026 17:17
Co-authored-by: Vadim Solovey <vadim@doit.com>

Co-authored-by: Codex <noreply@anthropic.com>
@chaim0m
chaim0m force-pushed the feat/chaim/pivot-visualization-CMP-48642 branch from 1224c20 to fc74750 Compare August 11, 2026 14:18
@chaim0m
chaim0m changed the base branch from feat/chaim/report-presentation-CMP-48642 to main August 11, 2026 14:18
@chaim0m
chaim0m merged commit 5fdf553 into main Aug 11, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-effort: 2 Low review effort

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants