search

bertt / mapbox-vector-tile-cs

A .NET library for decoding a Mapbox vector tile
MIT License
74 stars 14 forks source link

Support older versions with polygons having outer/inner ring NOT coded by winding order #7

Closed jonbe242 closed 7 years ago

jonbe242 commented 7 years ago

I have a version of Mapnik producing tiles with layers of version 1 of the spec, where polygons seems to be coded with outer ring first and any holes after that, i.e. NOT coded by winding order.

Version 2 of the spec first defined how to interpret which rings are exterior and which are interior. MVT renderers seems to be relaxed regarding how to interpret polygons of version 1. So I suggest this library do this to.

I can create a PR if you believe this is a wanted feature in your library.

bertt commented 7 years ago

Hi, in general I didn't want to support older versions of the spec, assuming tile generating tools will quickly support the newer specs. So why are you using this version of Mapnik? Or is version 1 the only supported spec there?

jonbe242 commented 7 years ago

We are using https://www.nuget.org/packages/mapnik which is quite old now, and we are stuck there until we find another way to render tiles from e.g. spatialite dbs from C# in a convenient way. I'd love to see a tile generator using NTS / GeoAPI or some other pure .NET lib.

bertt commented 7 years ago

ok clear, send the PR I'll take a look

bertt commented 7 years ago

closing this issue because no activity