mapbox/geojson-vt

Add an option to auto-assign unique ids to features

ryanbaumann opened this issue · 4 comments

ryanbaumann commented

Problem

GL JS v0.46.0 launched the setFeatureState() API, which allows a feature to be updated at run-time using the tile feature index. A feature index is not required in geojson sources, so a developer must add a geojson id value to their source before passing the data to geojson-vt.

Proposed Solution

  • Auto-generate a vector tile feature id for all geojson sources.

cc/ @mourner

ryanbaumann commented

Linking to mapbox/mapbox-gl-js#6849 as the GL-JS specific issue.

mourner commented

@ryanbaumann did you mean to post clustering-related improvements to the supercluster repo? geojson-vt only deals with unclustered sources.

ryanbaumann commented

@mourner yes, thanks for the catch! Supercluster issue -> mapbox/supercluster#88

Please leave this open if we would potentially handle automatic geojson feature id creation in geojson-vt for non-clustered sources. Otherwise, feel free to close.

mourner commented

Done in #109