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

Pie chart tooltip "Other" group produces NaN% #42458

Closed
alxnddr opened this issue May 9, 2024 · 0 comments · Fixed by #42465
Closed

Pie chart tooltip "Other" group produces NaN% #42458

alxnddr opened this issue May 9, 2024 · 0 comments · Fixed by #42465
Assignees
@EmmadUsmani
Labels
.Frontend .Team/DashViz Dashboard and Viz team Type:Bug Product defects Visualization/Charts/Pie
Milestone
0.49.10

Comments

@alxnddr
Copy link
Member

alxnddr commented May 9, 2024 •
edited

Describe the bug

Pie chart tooltip "Other" group produces NaN% and does not show values. Caused by #39180

Screenshot 2024-05-09 at 11 35 42 AM

To Reproduce

  1. New -> Orders -> Summarize by Total: 50 bins
  2. Hover the "Other" slice
  3. The tooltip shows NaN

Expected behavior

No response

Logs

No response

Information about your Metabase installation

Latest master

Severity

Makes it really hard to see what is grouped in the "Other" slice
@cbalusek cbalusek changed the title Pie chart tooltip "Other" group produces Pie chart tooltip "Other" group produces NaN% May 9, 2024
@EmmadUsmani EmmadUsmani mentioned this issue May 9, 2024
Merged
1 task
@EmmadUsmani EmmadUsmani self-assigned this May 9, 2024
@sloansparger sloansparger added this to the 0.49.10 milestone May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EmmadUsmani EmmadUsmani

Labels
.Frontend .Team/DashViz Dashboard and Viz team Type:Bug Product defects Visualization/Charts/Pie
Projects
None yet
Milestone
0.49.10
Development

Successfully merging a pull request may close this issue.

fix pie chart tooltip metabase/metabase
3 participants
@sloansparger @alxnddr @EmmadUsmani