search

Turfjs / turf-intersect

find the intersection of features
30 stars 9 forks source link

MultiPolygon coverage #13

Closed veltman closed 6 years ago

veltman commented 9 years ago

Is there any interest in a pull request that would allow for MultiPolygons as input/output? I'd guess a lot of people are writing the same "loop through the input polygons, loop through the comparison polygons, combine all the shards of overlap" code.

It's not super-complicated to wrap this with that extra code but it seems like it would make sense to include it here unless there's a design reason I'm missing for limiting it to polygons.

tcql commented 9 years ago

@veltman thanks for the interest in contributing. we're working on a rewrite of the turf-intersect, turf-union, and turf-difference internals, which will take care of this. This is still a little ways off, so if you had a need for multipolygon support in the near future, go ahead and write the wrapping :)

franckl commented 8 years ago

hi @tcql , is your team still working on a rewrite ?

valentinbdv commented 7 years ago

Hi @tcql, awesome library but I am missing this improvement too!

mykhas commented 6 years ago

Hi @tcql, I also need this improvement. Could I work on PR to fix this one?

DenisCarriere commented 6 years ago

MultiPolygon support has been added to @turf/intersect.