Shapely cascaded union should be replaced with unary union

aspectumapp / osm2geojson

Convert OSM and Overpass XML/JSON to GeoJSON

MIT License

100 stars 14 forks source link

Shapely cascaded union should be replaced with unary union #29

Closed ajcost closed 2 years ago

ajcost commented 2 years ago

Hi,

First off, love the package, super helpful. However it seems that the to geojson is still using cascaded union rather than unary union. Shapely has deprecated the use of unary union.

[osm2geojson/main.py:485](): ShapelyDeprecationWarning: The 'cascaded_union()' function is deprecated. Use 'unary_union()' instead.

Lines:

485, 475 of the main.py are affected.

ajcost commented 2 years ago

@rapkin