Option for SuperClusterer algorithm bounding box about js-markerclusterer HOT 6 CLOSED

That's an interesting point, we directly assumed it was a bug because of the browser crash and change in src/algorithms/supercluster.ts:96.

We have made another algorithm as suggested.

I'm interested in you mentioning that the current implementation is more optimized for concentrated data, because, that represents our case.
We were confronted with huge performance issues, using 2.3k markers in an approx. area of 30 km².

If you also feel divided by this, and understand the need for a dynamic bounding box in some cases, maybe it would be appealing to introduce it as a SuperClusterOptions?
(This goes hand-in-hand with extendBoundsToPaddedViewport(), so it doesn't feel too far-fetched to add it here.)

Some sort of :

enum bboxStatus {
    WHOLE_GLOBE,
    DYNAMIC,
    CUSTOM
}

function defineBoundingBox(status: bboxStatus = bboxStatus.WHOLE_GLOBE, custom?: number[]): number[] { ... }

If it were to be implemented and added in the docs, it would save lots of time for future people encountering this problem.

from js-markerclusterer.

🎉 This issue has been resolved in version 2.4.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)
• v2.4.0

Your semantic-release bot 📦🚀

from js-markerclusterer.

There is a trade off here that needs to be analyzed further.

• The current implementation only computes clusters when the zoom has changed but for the entire globe.
• The implementation in #135 computes clusters every time the map bounds changes but for a smaller region.

Which is better? Might depend on the specifics of the data. If markers are concentrated in a small area, the current implementation is probably more performant. If the markers are more uniform, probably the latter implementation is better.

It is possible to pass your own algorithm to meet your specific data patterns.

from js-markerclusterer.

Good suggestion on the configuration option.

from js-markerclusterer.

Hi, i have the same performance issue... it would help if this option existed in the supercluster options...
Do you plan to add this option in the next future version? thank you

from js-markerclusterer.

Please submit a pr! 😄

from js-markerclusterer.