TerriaJS / terriajs

A library for building rich, web-based geospatial data platforms.
https://terria.io
Apache License 2.0
1.19k stars 363 forks source link

Fix protomaps-leaflet (MVT) vertex limit #7240

Open nf-s opened 3 months ago

nf-s commented 3 months ago

Here we have MapboxGL on the left, and TerriaJS on the right (with protomaps version 1.19.0)

image

The latest version of protomaps-leaflet has fixed this issue - but we are using an older version

image

I believe this is the issue

We are currently upgrading protomaps - but it has taken a while due to upstream deprecations.

Currently, the only workaround is to reduce the complexity of the feature by reducing the number of vertices. Sorry I am unable provide a number of the maximum number of vertices - as the limit is based on the tiling (which happens on-the-fly in TerriaJS).