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

Multipolygon rendering incorrectly #11583

Closed
mfarmer-ara opened this issue Oct 24, 2023 · 2 comments · Fixed by #11577
Closed

Multipolygon rendering incorrectly #11583

mfarmer-ara opened this issue Oct 24, 2023 · 2 comments · Fixed by #11577

Comments

@mfarmer-ara
Copy link

Dragging and dropping a geoJSON file with multipolygon does not render as expected. It does render correctly when draggint the file onto geojson.io

However, looking at older versions of cesium it does render it correctly. I checked 1.92 and 1.108 and it rendered correctly in those.

Correctly rendering on version 1.92 (and up through 1.108)
jsFiddle1-92

Odd rendering on latest version on Sandcastle
sandCastle1-110

File:
MultiPolygonExample.json

Sandcastle example with drag and drop: https://sandcastle.cesium.com/#c=bZAxT8MwEIX/yilLE1E5YqVpBGq7FTEgMWVx7Ws54Z6js5O0IP47TtMBQT35nt/37snGc4jQEw4osATGAVYYqDuqt4uWN5m5zCvPUROjNFmxaLjhiVF4isg2v0KTuBZ9WItvn+lEPLqvXpu0jYgXpa3d9MhxSyHhac2+YxPJcz5Zn+Qwh+A7MTgHHJECvhqGdAZi6welHUrMZ5c4aMWnloH4ADO4u4LpMnsYx4lPNb6LRTbPqhDPDuspDeCRjq2XCJ24XKky4rF1OmIod535wKhMCCM7WqvyN1pZ6oHs8sYPgXE6hPSy75x7pU9ssroqk/8f6ry2qfZLj+L0ebS939fbSVRKVWUab5PRe7fT8if5Bw

JSFiddle examples of cesium with drag and drop:
1.92 https://jsfiddle.net/Lj5wbz38/
1.108 https://jsfiddle.net/gdfmaL9n/
Select "Natural Earth II" to get the map, then drop the file on.

Browser: Edge 118

Operating System: Windows 10

Also tested on Ubuntu 20, Chrome version 118

@ggetz
Copy link
Contributor

ggetz commented Oct 24, 2023

Thanks for the report @mfarmer-ara! This appears to be fixed in #11577. I would expect that fix to go out with the next release.

@mfarmer-ara
Copy link
Author

@ggetz Fantastic, I'll check again with the next release.

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