search

mapbox / tile-cover

Generate the minimum number of tiles to cover a geojson geometry
MIT License
189 stars 40 forks source link

Is different min/max_zoom useful? #46

Closed mourner closed 9 years ago

mourner commented 9 years ago

@morganherlocker @yhahn to optimize tile-cover further, I need to understand the use cases better... Is there an actual use case for supplying different min_zoom and max_zoom? I see that it's not used in carmen... The performance of merging siblings is HORRIFYING:

tilecover.tiles(russia, {min_zoom: 9, max_zoom: 10});

image

It's horrifying to the point it looks easier to just remove the functionality and forget about it (if there are no good use cases) than try to optimize.

morganherlocker commented 9 years ago

@mourner Yes, there are cases. If tile-cover is used as a spatial database index processor, a user would want to minimize the number of tiles in the index (less storage used, less reads, etc.). In these cases, an insert perf hit could be worth a big select perf boost.

mourner commented 9 years ago

Right. Tracking performance in #49