Skip to content

ScatterGL Hover Missing /w Data Gaps #2524

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

Closed
Kolano opened this issue Apr 4, 2018 · 4 comments
Closed

ScatterGL Hover Missing /w Data Gaps #2524

Kolano opened this issue Apr 4, 2018 · 4 comments
Labels
bug something broken

Comments

@Kolano
Copy link

Kolano commented Apr 4, 2018

On ScatterGL plots with gaps in their data (i.e. null values on Y axis array) hover pop-ups only show up for the right-most (last?) data points. This seems to be handled correctly with the Scatter type.

@etpinard
Copy link
Contributor

etpinard commented Apr 4, 2018

Thanks for the report! Can you please a reproducible example to help us out?

@Kolano
Copy link
Author

Kolano commented Apr 4, 2018

I tried to provide a Chart Studio link for it, but is seems Chart Studio may not use the GL versions of plots, so here's a CodePen...
https://codepen.io/Kolano/pen/ZxMKza

@Kolano Kolano changed the title ScatterGL Hover and Gaps ScatterGL Hover Missing /w Data Gaps Apr 4, 2018
@alexcjohnson
Copy link
Collaborator

Interesting - it depends on the length of the gap, or perhaps the total number of missing values? See https://codepen.io/alexcjohnson/pen/NYLaPR - at the end I have a loop that replaces null with 0 up to a certain point (for(var i = 0; i < fig.data[0].y.length/1.8; i++)) - at 1.8 the problem is still there (and in fact some of the 0 values before (to the right of) the gap also have no hover) but if you decrease that number to 1.75 or less it all works as expected.

@alexcjohnson alexcjohnson added the bug something broken label Apr 4, 2018
@etpinard
Copy link
Contributor

etpinard commented Apr 6, 2018

This appears fixed in master:

peek 2018-04-06 16-40

most likely due to @dy's point clustering improvements in #2499


So ... I'll close this thing. @Kolano feel free to open up a new issue if this bug persist in 1.36.0 once it is released (about ~10 days from now).

@etpinard etpinard closed this as completed Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

No branches or pull requests

3 participants