gdal icon indicating copy to clipboard operation
gdal copied to clipboard

ogr2ogr: speed-up -clipsrc/-clipdst, and add -skipinvalid

Open rouault opened this issue 1 year ago • 1 comments

  • ogr2ogr: speed-up -clipsrc/-clipdst by avoiding GEOS when possible

    If the envelope of the feature's geometry is contained in the envelope of the -clipsrc/-clipdst geometry, we can just avoid any GEOS operation.

  • ogr2ogr: add -skipinvalid to skip features whose geometry is not valid w.r.t Simple Features

rouault avatar Jun 29 '24 18:06 rouault

Coverage Status

coverage: 69.262% (+0.001%) from 69.261% when pulling 3d6d6d45fd8361fb087152a3938ef6b3b2cca953 on rouault:ogr2ogr_clipsrc_dst_speedup into 3e41319ccb7b8b8b3a35db07553fd1eae6249d7f on OSGeo:master.

coveralls avatar Jun 29 '24 18:06 coveralls

Note: the first commit in this pull request is erroneous. Fixed per https://github.com/OSGeo/gdal/pull/11655

rouault avatar Jan 15 '25 00:01 rouault