[Android]Map crash when adding multiple pins if clustering is enabled

TorbenK / TK.CustomMap

Extended Map Control

MIT License

142 stars 109 forks source link

[Android]Map crash when adding multiple pins if clustering is enabled #319

Open jeremymarabel opened 6 years ago

jeremymarabel commented 6 years ago

Hi,

With clustering enabled, the map crash when we try to add multiple pins. (Only with android, iOS works great) I dont get any stacktrace except this : [] * Assertion at /Users/builder/jenkins/workspace/xamarin-android-d15-6/xamarin-android/external/mono/mono/metadata/threads.c:3016, condition 'internal' not met [libc] Fatal signal 6 (SIGABRT), code -6 in tid 13319 (Thread-644)

As it is not a very helpfull stacktrace, i created a small project to demonstrate the problem. Simply launch the sample and add more pins via the toolbar button. (Don't forget to put you api key in the android manifest) If you need more informations feel free to ask.

Thanks for your time, hope this can be fix easily.

Archive.zip

Knots23 commented 6 years ago

Encountered the same issue. I've tried to:

Set the clustering enabled, then add multiple pins to the Pins collection.
Set the clustering disabled, then add all pins to the Pins collection, wait an arbitrary amount of time (2 seconds), then enabled the clustering.
Create a new ObservableCollection of pins and set the Pins property anew (instead of a clear, followed by adding new pins to the collection), then set the IsClusteringEnabled to true.

All of the above leads to the exception mentioned by @jeremymarabel

If this worked before, could you please check where it went wrong in the projects history? Given the nature of the exception, I'm thinking an update of some Xamarin NuGet package.

In its current state, the package can be used, as long as the clustering remains disabled.

DanielD commented 6 years ago

Also having this issue. I disabled clustering, but it is very difficult to select the pins when they are all on top of each other.

ssrodnas commented 5 years ago

Hi, do you have any update related to this issue?

jeremymarabel commented 5 years ago

No sorry, ended up implementing clustering myself. Basically I just have a different list of pins that i add or remove to the map depending of the zoom level, with custom pin showing design the number of pin clustered.

ssrodnas commented 5 years ago

Ok, thanks! So you implemented the clustering and doing the clustering on every zoom level changes?