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

Bar text performance regression from plotlyjs 1.58.4 to 2.1 #5764

Closed
darintay opened this issue Jun 24, 2021 · 5 comments
Closed

Bar text performance regression from plotlyjs 1.58.4 to 2.1 #5764

darintay opened this issue Jun 24, 2021 · 5 comments

Comments

@darintay
Copy link

Bar charts with text appear to be several times slower to render in plotlyjs 2.1 than they were in 1.58.4.

Simple example here:

1.58.4: https://jsbin.com/winiqazavu/edit?html,output
2.1.0: https://jsbin.com/fahedevopo/edit?html,output

The 1.58.4 version runs in a couple of seconds for me, where the 2.1.0 takes over a minute. I've tried on Chrome and Firefox.

If you remove the text, everything renders quickly on either version.

@sean-mcl
Copy link

Hey, we found some performance issues too.

Here is an example with 100k datapoints. When we use the 2.2.0, the whole plot freezes and has a very bad performance. When we use the plotly-1.58.4.min.js, everything works fine.

LayTec-AG/Plotly.Blazor#99

@archmoj
Copy link
Contributor

archmoj commented Jun 30, 2021

This is related to #5624 and #5638.
Please try setting textposition to "none" in your bar trace.

@sean-mcl
Copy link

@archmoj Can you please check the example above regarding the performance of the scallergl plot.

@archmoj
Copy link
Contributor

archmoj commented Jun 30, 2021

@archmoj Can you please check the example above regarding the performance of the scallergl plot.

Please open a separate issue for the scattergl case.
Thanks.

@archmoj archmoj closed this as completed Jun 30, 2021
@DarinTayTwoXAR
Copy link

Ah, it was suddenly trying to render the text inside all the bars? Makes sense why that would take a while.

Confirmed that textposition none fixes issues here, thanks!

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