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

Better big data support for scattermapbox #1885

Closed
jackparmer opened this issue Jul 17, 2017 · 5 comments
Closed

Better big data support for scattermapbox #1885

jackparmer opened this issue Jul 17, 2017 · 5 comments
Labels
feature something new
Milestone

Comments

@jackparmer
Copy link
Contributor

jackparmer commented Jul 17, 2017

We're getting Dash apps through the roof that use scattermapbox. Would be nice to see how far we can push the # of points with a WebGL overlay instead of an SVG one to improve big data support for scattermapbox traces.

Maybe we could investigate: https://github.com/vicapow/react-map-gl-stack-gl-overlay-example

This plot of 🌳 s in Manhattan for example is starting to push the mapbox-gl limits:

https://plot.ly/~jackluo/444

/cc @jackwluo @dfcreative

@jackparmer jackparmer added the feature something new label Jul 17, 2017
@jackparmer jackparmer added this to the Dash milestone Jul 17, 2017
@etpinard
Copy link
Contributor

I think you mean the mapbox-gl limits? scattermapbox renders all things in using mapbox-gl which uses purely WebGL. No SVG whatsoever here. Editing ⤴️ accordingly.

I think we should investigate other pure mapbox-gl solutions before doing this:

  1. find what are those pure mapbox-gl limits i.e. try to plot the as many as possible using only mapbox-gl (not via the plotly.js wrapper)
  2. plotly.js uses [email protected], which more than 10 minor versions behind the mapbox-gl latest, so we should test bleeding edge mapbox-gl to see if the rendering better
  3. try mapbox-gl clustering.

@etpinard etpinard changed the title scattermapboxgl Better big data support for scattermapbox Jul 17, 2017
@immotus
Copy link

immotus commented Oct 16, 2017

Are there any plans to update mapbox-gl dependency to the latest version?

@rreusser
Copy link
Contributor

@PavelGolodoniuc See #2029 to track progress on that.

@etpinard
Copy link
Contributor

etpinard commented Feb 7, 2018

Performance for

Plotly.d3.json('https://plot.ly/~jackluo/444.json', (err, fig) => {
    delete fig.layout.mapbox.accesstoken
    Plotly.newPlot(gd, fig)
})

is pretty solid using [email protected] 🎉

@etpinard
Copy link
Contributor

etpinard commented Feb 13, 2018

#2361 will improve scattermapbox performance significantly.

Closing this thing (for now).

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

No branches or pull requests

4 participants