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

P0: Oddly drawn polygons #79

Closed
seasmith opened this issue May 31, 2018 · 5 comments
Closed

P0: Oddly drawn polygons #79

seasmith opened this issue May 31, 2018 · 5 comments

Comments

@seasmith
Copy link

There are at least three oddly drawn polygons (US counties) in the image below: one in Alaska and two in the contiguous 48 states. This was produced with the kepler.gl website.

The was geojson file was originally an sf object stored in a .RData file (R language) - subsequently converted to geojson.

I attempted to replicate this problem with the geojson file in D3 v4 (seen here), but it did not produce the same oddly drawn polygons.

I have the geojson file, kepler config file, and D3 attempt contained within this gist.

@rsimon
Copy link

rsimon commented Jun 5, 2018

I'm seeing the same issue, although on a more massive scale :-)

image

I also used the kepler.gl Website. Uploaded the data as a CSV, with a geometry column containing WKT.

To keep things small, I've pasted a CSV with a single example (shown below) in this gist. If it's helpful, I'm happy to share the full CSV. Just let me know!

image

image

@heshan0131
Copy link
Contributor

This should be fixed once we upgrade to deck.gl v5

@heshan0131 heshan0131 changed the title Oddly drawn polygons P0: Oddly drawn polygons Jun 5, 2018
@guidorice
Copy link

@heshan0131 I have noticed this too with some geojson files. Is this because of the polygon winding order, or something else? I tried running https://github.com/mapbox/geojson-rewind to fix a file, but still get the same rendering problem.

btw thanks for confirming a fix is coming in deck.gl 5 👍

@palamago
Copy link

Indeed. Fixed with latest version of deck.gl. You can close this issue.

@seasmith
Copy link
Author

Fixed with deck.gl v5.

bjungs pushed a commit to imec-int/kepler.gl that referenced this issue Nov 14, 2022
…er-layer

[PREC-94] add wmts layers to simulations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants