googlemaps / googlemaps/android-maps-utils
Relax generic requirement on ClusterItem on clustering related interfaces
- Dominant language
- Kotlin
- Stars
- 3.6k
- Forks
- 1.6k
- Avg merge
- 2d 20h
- Merged PRs (30d)
- 17
Description
Thanks for stopping by to let us know something could be better!
---
**PLEASE READ**
If you have a support contract with Google, please create an issue in the [support console](https://cloud.google.com/support/). This will ensure a timely response.
Discover additional support services for the Google Maps Platform, including developer communities, technical guidance, and expert support at the Google Maps Platform [support resources page](https://developers.google.com/maps/support/).
If your bug or feature request is not related to this particular library, please visit the Google Maps Platform [issue trackers](https://developers.google.com/maps/support/#issue_tracker).
Check for answers on StackOverflow with the [google-maps](http://stackoverflow.com/questions/tagged/google-maps) tag.
---
**Is your feature request related to a problem? Please describe.**
Currently the library requires you to implement `ClusterItem` on your data classes. This can be problematic when you don't own those classes and/or producers of the data (or just don't wan't your model to depend on this particular library).
**Describe alternatives you've considered**
Creating wrapper implementing `ClusterItem`. This solution gets slow with large datasets/fast changing data. Additionaly it's just plain waste of memory.
**Describe the solution you'd like**
Remove the `extends ClusterItem` from interfaces like `ClusterRenderer` and `Algorithm` and thus in unopinionated manner enable other techniques of handling this problem.
Contributor guide
Assessment
This issue has not been assessed yet.