rill icon indicating copy to clipboard operation
rill copied to clipboard

APP-119 feat: comparison in canvas charts

Open djbarnwal opened this issue 1 month ago • 4 comments

When comparison time range is enabled add comparison to -

  • Line chart
  • Stacked bar chart
  • Grouped bar chart
  • Stacked normalized bar chart
  • Multi measure charts (all above types)

https://linear.app/rilldata/issue/APP-119/time-comparisons-on-canvas-charts

Checklist:

  • [ ] Covered by tests
  • [x] Ran it and it works as intended
  • [x] Reviewed the diff before requesting a review
  • [x] Checked for unhandled edge cases
  • [x] Linked the issues it closes
  • [ ] Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • [ ] Intend to cherry-pick into the release branch
  • [ ] I'm proud of this work!

djbarnwal avatar Nov 08 '25 12:11 djbarnwal

So it looks like comparison only works for all chart types except donus, funnel, heatmap, and combo chart?

Also noticed that local time range comparison is only available on KPI grids, Tables, Leaderboards, but not charts?

ericokuma avatar Nov 25 '25 22:11 ericokuma

So it looks like comparison only works for all chart types except donut, funnel, heatmap, and combo chart?

Yes, not all chart types have a standard complimentary comparison version. If needed, we can think about comparison would look like for these charts and implement it as a follow up.

Also noticed that local time range comparison is only available on KPI grids, Tables, Leaderboards, but not charts?

Good catch, added it for chart types that support comparison.

djbarnwal avatar Nov 25 '25 23:11 djbarnwal

LGTM!

ericokuma avatar Nov 26 '25 00:11 ericokuma

@briangregoryholmes, when you get a chance, can you do a code review here? Thanks in advance!

ericokuma avatar Dec 08 '25 18:12 ericokuma