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.

Sign up for GitHub

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

Jump to bottom

KML renders incorrectly - vertical polygons #3349

Closed
mramato opened this issue Dec 15, 2015 · 5 comments
Closed

KML renders incorrectly - vertical polygons #3349

mramato opened this issue Dec 15, 2015 · 5 comments
Labels
category - kml type - bug

Comments

@mramato
Copy link
Contributor

mramato commented Dec 15, 2015

The attached KML does not render correctly because it contains polygons that hare mostly perpendicular to the surface of the earth.

2_1450116435.kml.txt
@mramato
Copy link
Contributor Author

mramato commented Dec 15, 2015

Reported via the forum: https://groups.google.com/d/msg/cesium-dev/p2VeANQ497I/PvXjDXpJBgAJ

@pjcozzi pjcozzi mentioned this issue Dec 15, 2015
Open
30 tasks
@hpinkos hpinkos mentioned this issue Oct 17, 2017
Closed
@hpinkos hpinkos changed the title KML renders incorrectly KML renders incorrectly - vertical polygons Oct 17, 2017
@hpinkos
Copy link
Contributor

hpinkos commented Oct 17, 2017

See another example here: #5907 (comment)

@ggetz ggetz mentioned this issue Mar 15, 2018
Closed
@hpinkos hpinkos mentioned this issue Jun 6, 2018
Closed
@hpinkos
Copy link
Contributor

hpinkos commented Jun 6, 2018

Also reported by @silky in #6662

@hpinkos hpinkos mentioned this issue Jul 3, 2018
Merged
@hpinkos
Copy link
Contributor

hpinkos commented Jul 3, 2018

@bmckilligan @silky I just opened this PR for adding support for vertical polygons #6769

@hpinkos hpinkos mentioned this issue Jul 10, 2018
Merged
2 tasks
@cesium-concierge
Copy link

Congratulations on closing the issue! I found these Cesium forum links in the comments above:

https://groups.google.com/d/msg/cesium-dev/p2VeANQ497I/PvXjDXpJBgAJ

If this issue affects any of these threads, please post a comment like the following:

The issue at #3349 has just been closed and may resolve your issue. Look for the change in the next stable release of Cesium or get it now in the master branch on GitHub https://github.com/AnalyticalGraphicsInc/cesium.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category - kml type - bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pjcozzi @mramato @hpinkos @cesium-concierge