-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add clustering options to scattermapbox
#5827
Conversation
Add clustering to scattermapbox
cluster: { | ||
enabled: { | ||
valType: 'boolean', | ||
dflt: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we use a smart default here? ie true
if any other cluster
attributes are set, otherwise false
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call. Done in 477d2a0.
Co-authored-by: Alex Johnson <alex@plot.ly>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💃 LGTM!
…y.js#5827), add top level args for clustering
Follow up for PR #5334.
Options added by @elben10 to enable mapbox clustering layer.
Please note that currently it doesn't support the fill, selection and text features.
Some remaining TODOs:
Plotly.react
can switch between modes i.e. with/withoutcluster
step
andopacity
@plotly/plotly_js