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.

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

scrollZoom performance issues since 2.0.0-rc.2 #6054

Closed
secretwpn opened this issue Dec 9, 2021 · 5 comments
Closed

scrollZoom performance issues since 2.0.0-rc.2 #6054

secretwpn opened this issue Dec 9, 2021 · 5 comments

Comments

@secretwpn
Copy link

I don't have rights to reopen the #5912 so creating a follow-up issue

Here's a demo: https://codesandbox.io/s/plotly2-performance-hovermode-0hf9z?file=/src/index.js

It is set to run with Plotly 2.0.0-rc.1 and the zoom is smooth like butter
But change it to rc.2 or any fresher version and try zooming in with mouse wheel some good amount - after certain point it will become unusably sluggish

@archmoj
Copy link
Contributor

archmoj commented Dec 10, 2021

Please note that the default hovermode is 'closest' in v2.
Could you please try setting hovermode: 'x' in the layout?

@secretwpn
Copy link
Author

secretwpn commented Dec 10, 2021

@archmoj Yes, I'm aware of that and hovermode is explicitly defined in layout properties of my demo.

Also this is critical for my app to have hovermode: 'closest' and the point is - it did work with this exact hovermode just fine in rc.1

The demo also has an illustration on how performance differs with newer plotly version on different kind of data - randomized vs sampled from test system.

@shreeramk
Copy link

@secretwpn With scattergl and hovermode set to closest, I m seeing slowness as well.
@archmoj please help us here.

@archmoj
Copy link
Contributor

archmoj commented Jan 24, 2022

Here is changelog diff between rc1 and rc2:
c74be05

Could you please investigate if this is a side effect of #5632?

@gvwilson
Copy link
Contributor

Hi - we are trying to tidy up the stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for several years, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. Thanks for your help - @gvwilson

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants