Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KML clustering crash #4414

Closed
hpinkos opened this issue Oct 7, 2016 · 2 comments
Closed

KML clustering crash #4414

hpinkos opened this issue Oct 7, 2016 · 2 comments

Comments

@hpinkos
Copy link
Contributor

hpinkos commented Oct 7, 2016

Reported on the forum: https://groups.google.com/forum/?hl=en#!topic/cesium-dev/p1Mv6vpvvnY

If you use this kml file in the Sandcastle clustering example, it crashes. The KML has two placemarks in it

placenames_capital.kml.txt

@pjcozzi
Copy link
Contributor

pjcozzi commented Oct 15, 2016

@bagnell can you please look at this for 1.27 perhaps even at the bug bash?

@mramato
Copy link
Contributor

mramato commented Oct 21, 2016

The fix here appears to be to add defined(entityCluster._labelCollection) && to line 148 of EntityCluster.js, doing so appears to make everything work as expected. I tried (and failed) to create a unit test for this, so perhaps @bagnell can do it when he gets back on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants