added bbox test

[tcql]

querying with a bbox returns properties for all polygons that it touches.

If there's only one hit, just the match is returned instead of an array.

still exposed the rbush tree, in case anybody using which-polygon requires additional access.

[mourner]

This may be a bit confusing since you do checks against geometry in polygon tests, but don't do it with bbox. So maybe it's not worth exposing as a public query method, because users would expect it to behave properly.

[tcql]

oh good point. I kind of spaced out and threw this together real fast. We could use lineclip to figure out if the polygons actually intersect (or use point in polygon to detect if the bbox is entirely inside a polygon)

[tcql]

@mourner updated to use lineclip to validate matches

[mourner]

Let's use a simple loop like in query instead of two filter/map closures for consistency.

[tcql]

updated 👍

[mourner]

@tcql @ingalls Rebased against master and was about to merge, but just realized that the bbox check is not always correct. One counter-case that comes to mind is when bbox center is inside a hole which is fully contained within the bbox. Then bbox intersects the polygon, while the outer ring doesn't intersect the bbox. This needs another thought...

[mourner]

Fixed the edge case above and published as 2.1.0.

[ingalls]

Thanks a ton @mourner ! Will get the mbtiles-extrats PR updated to point at this version.