Skip to content

Fast scroll zoom results in out of memory error #3615

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
arnog opened this issue Mar 7, 2019 · 7 comments
Closed

Fast scroll zoom results in out of memory error #3615

arnog opened this issue Mar 7, 2019 · 7 comments
Labels
bug something broken

Comments

@arnog
Copy link

arnog commented Mar 7, 2019

See https://codepen.io/arnog/pen/YgZdEZ

  1. Create a heatmap plot with about 500 x 500 elements
  2. Configure the plot with scrollZoom = true and zsmooth = 'fast'
  3. Use the mouse wheel to zoom in rapidly into the plot

Result:

  • the plot is not zoomed in and the console displays "Uncaught RangeError: Invalid array length"

The array being created is the pixel array in which to draw the interpolated heatmap at traces/heatmap/plot.js:238

The problem seems to be that imageWidth and imageHeight are used here and later in the function, perhaps instead of canvasW and canvasH.

@antoinerg
Copy link
Contributor

@arnog I could not replicate the particular error message you received. What operating system and web browser were you using?

@archmoj Can you reproduce this issue?

@antoinerg
Copy link
Contributor

While investigating the issue reported by @arnog, I uncovered the following bug in heatmapgl: #3624

@alexcjohnson
Copy link
Collaborator

I managed to crash the browser tab zooming in on this plot. It got slower and slower as I zoomed in, eventually crashing. But I also see a bug when you zoom out, an increasingly small fraction of the total trace is shown.
heatmap scroll sm
Mac OS 10.14, Chrome 72.

@antoinerg
Copy link
Contributor

antoinerg commented Mar 11, 2019

I could reproduce the issue as well. When zooming in, the memory usage skyrockets from ~200M to a whopping 1500M.

NixOS 18.09, Chromium 69

Thanks @arnog

@gvwilson
Copy link
Contributor

Hi - this issue has been sitting for a while, so as part of our effort to tidy up our public repositories I'm going to close it. If it's still a concern, we'd be grateful if you could open a new issue (with a short reproducible example if appropriate) so that we can add it to our stack. Cheers - @gvwilson

@arnog
Copy link
Author

arnog commented Jun 11, 2024

The issue still happens. I will open a new issue: #7021

@gvwilson
Copy link
Contributor

thank you @arnog

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

4 participants