Add cluster features to scattermapbox trace#5147
Conversation
|
@elben10 thanks for the PR, it looks really useful! But let's take a step back before going further with it - does this really need to be a separate trace type? It seems to me as though, in terms of the attributes involved, it's just a few extra options on top of Does that seem like a reasonable way to include this feature? |
|
@alexcjohnson I think that sounds like a better choice. I'll update the code accordingly |
Co-authored-by: Mojtaba Samimi <33888540+archmoj@users.noreply.github.com>
- Remove radius attribute as it is replaced by the size attribute - Use plotly defaults in attributes
|
337 file changes! This is now extremely hard to review. |
|
@elben10 To open the final PR you may consider starting a new fork. So that you get credit for the great contribution you made. Then please open a PR from your Does that make sense? |
|
Yes it does. The pull request is now created! Thanks for the help and guidance! |
I've added a trace that allow one to use mapbox clustering layer.
Currently it doesn't support the fill, selection and text features.