iOS Example: Tapping "Reset" button results in another bunch of annotations added to the map without clearing the old ones. about cchmapclustercontroller HOT 6 CLOSED

This bug makes this library virtually unusable. Has there been a fix or at least a workaround for this yet?

from cchmapclustercontroller.

For me, this only happens when I quickly add and remove annotations at the same time to the clustering – which is a use case I don't have.

You can help by

• Creating a unit tests that reproduces this problem
• Provide a solution that avoids this issue

Looking forward to your contributions

from cchmapclustercontroller.

I can reproduce it consistently on the simulator iOS 9.2 (13C75). It seems to be a problem with MKPointAnnotation, which gets added to the set of annotations without detecting duplicates.

Overriding the hash and isEqual as I did in 539c98b seems to be fixing the problem.

from cchmapclustercontroller.

@bhapca Can you please let me know how to reproduce this issue? I'm running the sample app in this repo on Xcode 7.2.1/iPhone 6/iOS 9.2 and tap the reset button. Where does the crash occur for you?

from cchmapclustercontroller.

@choefele, I'm not experiencing any crash, instead I'm getting the duplicate annotations discussed in this issue. The screenshots I included show how the annotations set double in size after 2 consecutive taps on the Reset button. They're screenshots of the debug session, stopped at breakpoint, not crashed, (look at the lldb prompt) which shows the issue.

The steps to repro are the same as described by @grigorye at the beginning of this thread. Essentially launch the sample app, wait for the map to load, tap Reset, see how the number of annotations increase. Repeat tapping Reset, to see the number of annotations keeping growing.

It appears that the allAnnotations property of CCHMapClusterController doesn't behave like a set when it's fed instances of MKPointAnnotation, because it accepts duplicate entries.

It seems that the isEqual method of MKPointAnnotation is messed-up:

from cchmapclustercontroller.

I believe this is fixed in 674ec58

from cchmapclustercontroller.