Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tooltip shows incorrect values on unaggregated data with breakout #15785

Closed
alxnddr opened this issue Apr 23, 2021 · 2 comments
Closed

Tooltip shows incorrect values on unaggregated data with breakout #15785

alxnddr opened this issue Apr 23, 2021 · 2 comments
Assignees
@alxnddr
Labels
.Correctness Priority:P3 Cosmetic bugs, minor bugs with a clear workaround .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Charts Line, area, bar, combo, and scatter charts.
Milestone
0.39.1

Comments

@alxnddr
Copy link
Member

alxnddr commented Apr 23, 2021 •
edited
Loading

Describe the bug
When there are two or more columns with unaggregated data and breakouts the tooltip shows incorrect values.

To Reproduce
Steps to reproduce the behavior:

  1. Open SQL editor and paste the following query
select 1 as axis, 5 as value, 9 as breakout union all
select 2 as axis, 6 as value, 10 as breakout union all
select 2 as axis, 6 as value, 10 as breakout
  1. Create a bar chart with the following settings

Screen Shot 2021-04-23 at 20 37 52

** Actual result **
Screen Shot 2021-04-23 at 20 35 55

Expected behavior
The tooltip should show Value: 12

Information about your Metabase Installation:
0.39, but likely to exist for a long time

It is related to #11907 but reproduces only with breakouts
@alxnddr alxnddr added Type:Bug Product defects .Needs Triage labels Apr 23, 2021
@alxnddr alxnddr mentioned this issue Apr 23, 2021
Closed
@nemanjaglumac
Copy link
Member

nemanjaglumac commented Apr 23, 2021 •
edited
Loading

I tried to reproduce locally and this is a screenshot from a Cypress run. I'm seeing the correct value.
image

Also, cannot reproduce on release-x.39.x.

nemanjaglumac added a commit that referenced this issue Apr 23, 2021
@nemanjaglumac
Add repro for #15785
6211b68
@nemanjaglumac nemanjaglumac mentioned this issue Apr 23, 2021
Merged
@flamber flamber added .Correctness Priority:P3 Cosmetic bugs, minor bugs with a clear workaround Visualization/Charts Line, area, bar, combo, and scatter charts. and removed .Needs Triage labels Apr 26, 2021
nemanjaglumac added a commit that referenced this issue Apr 26, 2021
@nemanjaglumac
#15785 Repro: Tooltip shows incorrect values on unaggregated data wit…
f16c811 
…h breakout (#15788)
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label Apr 26, 2021
@alxnddr
Copy link
Member Author

alxnddr commented Apr 26, 2021 •
edited by flamber
Loading

Fixed by #15778

@alxnddr alxnddr closed this as completed Apr 26, 2021
@flamber flamber added this to the 0.39.1 milestone Apr 26, 2021
@deploysentinel deploysentinel bot mentioned this issue Dec 5, 2022
Merged
@deploysentinel deploysentinel bot mentioned this issue Jan 7, 2023
Closed
@deploysentinel deploysentinel bot mentioned this issue Oct 25, 2023
Closed
1 task
This was referenced Dec 2, 2023
Merged
Merged
Merged
This was referenced Jan 11, 2024
Merged
Merged
Merged
Merged
Merged
Closed
Merged
This was referenced Feb 5, 2024
Merged
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Open
Merged
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alxnddr alxnddr

Labels
.Correctness Priority:P3 Cosmetic bugs, minor bugs with a clear workaround .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Charts Line, area, bar, combo, and scatter charts.
Projects
None yet
Milestone
0.39.1
Development

No branches or pull requests
3 participants
@flamber @alxnddr @nemanjaglumac