googlemaps / googlemaps/android-maps-utils

Relax generic requirement on ClusterItem on clustering related interfaces

Open
#779 24 comments 1 reaction 1 assignee Claimed by @kikoso View on GitHub
priority: p2 semver: major type: feature request
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.