-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
wrong range in hover info for basic histogram #5848
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.
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
Comments
This is all in service of greater clarity at the bin edges. To be precise, what's happening here is two things:
What we really DON'T want to do is have labels 50-100 and 100-150, because then it's ambiguous in which bin we put a value of exactly 100. But you could perhaps argue that the bin shift should match the range shrinkage - ie because we shifted the bins exactly 0.5 here we should also shrink the ranges we report by exactly 0.5 on each side, to 50-99, or if we want to keep 50-90 we should shift the bins by 5. |
I could also see us listing the bounds of the data in each group, or using |
Thanks for commenting! I actually think labels I don't think giving the range of the observations (e.g., To me, the optimal solution is |
Right, so having thought about this more, as Alex says, the bin bounds aren't actually 50-100 here, they're 49.5-99.5 (due to the smart behaviour around integers). We can maybe debate later if this is a good idea, but at the very least we should frame our discussion around the actual bounds :) I think then that for this specific chart the hover should be I should note that forcing the bins to 50-100 (verified by zoom!) with
I don't think that's confusing, personally, I think it's clarifying :) |
Hi, I'm just trying to bring this issue/bug up again, as having correct hover information on the length of the bins would be a great enhancement for histograms. |
The hover information on the bin range shown for this basic histogram is rather misleading:
I would have expected a range of "50 - 100" shown for the second bin. (The same mislabeling occurs for all other bins.)
Codepen:
https://codepen.io/bklingenberg/pen/zYwdaOq?editors=0011
(This may be relate to discussions in #2113.)
The text was updated successfully, but these errors were encountered: