@turf/voronoi - use d3-delaunay in favor of d3-voronoi

Turfjs / turf

A modular geospatial engine written in JavaScript and TypeScript

https://turfjs.org/

MIT License

9.19k stars 934 forks source link

@turf/voronoi - use d3-delaunay in favor of d3-voronoi #2220

Open gavinr opened 2 years ago

gavinr commented 2 years ago

I noticed that turf-voronoi uses d3-voronoi, but that project is now archived, and it suggests to use d3-delaunay.

I just wanted to see if it's possible to switch over? And is are the maintainers open to a PR that does that?

Thanks!

(note this was originally added by @stevage in #1043) (possibly related to #1053)

maral commented 1 year ago

Is it okay that d3-delaunay uses iterators, which are not available in ES5? I could do PR if it is okay.

maral commented 1 year ago

Scratch that, for WSG coordinates, d3-geo-voronoi should work better and would solve #1053 . The usage of d3-geo-voronoi was suggested in #1043 but was never implemented.