CesiumGS / CesiumGS/cesium

Fill Complex Polygons

Open
#1,209 10 comments 0 reactions 0 assignees View on GitHub
category - polygons/geometry type - enhancement
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

As originally reported in #1121 and started in #1163.

A few test files:
- [ne_10m_us_states.json](https://github.com/AnalyticalGraphicsInc/cesium/blob/3e7a34dcda91f01f41be71c6d3fd5fcb85aa0166/Apps/CesiumViewer/Gallery/ne_10m_us_states.json)
- [NO_Admin_latlng.topojson](https://raw.github.com/turban/N2000-Kartdata/master/NO_Admin_latlng.topojson)
- [HammontonOnlyGeo2.topojson](http://gis.civilsolutions.biz/geojson/HammontonOnlyGeo2.topojson)

In particular, ne_10m_us_states.json has a complex polygon on the southeast coast of Texas just south of Sweeny.

![image](https://f.cloud.github.com/assets/782098/1098436/32b07c92-1721-11e3-95a3-f08e8ebaf832.png)

Zoomed out:
![image](https://f.cloud.github.com/assets/782098/1098440/3ea0a964-1721-11e3-8000-a49e28cde08b.png)

Contributor guide

Open the contributing guide

Research direction

Review issues #1121 and #1163 first, then reproduce the polygon-rendering problem with ne_10m_us_states.json, especially the complex polygon south of Sweeny, Texas. Compare the result with NO_Admin_latlng.topojson and HammontonOnlyGeo2.topojson; done means these complex polygons render correctly without the reported artifacts.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
computer-graphics
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.