Opened 7 years ago

Closed 6 years ago

#3809 closed enhancement (wontfix)

Add ST_DistanceIndexed function

Reported by: dbaston Owned by: dbaston
Priority: medium Milestone: PostGIS 3.0.0
Component: postgis Version: master
Keywords: Cc:

Description

Add an ST_DistanceIndexed function that delegates to the IndexedFacetDistance implementation in GEOS. In the future, this may be replaced with a native LWGEOM implementation.

Change History (8)

comment:1 by dbaston, 7 years ago

Type: defectenhancement

comment:2 by dbaston, 7 years ago

The indexed distance calculations in GEOS use a particular definition of "distance." From the JTS docs:

Distances will be computed to all facets of the input geometry. The facets of the geometry are the discrete segments and points contained in its components. In the case of Lineal and Puntal inputs, this is equivalent to computing the conventional distance. In the case of Polygonal inputs, this is equivalent to computing the distance to the polygons boundaries.

When one polygon is entirely contained within another polygon, the distance between them is greater than zero.

I think this functionality is valuable in PostGIS, but the name ST_DistanceIndexed risks creating confusion.

Is one of these options better?

  • ST_LinearDistanceIndexed
  • ST_BoundaryDistanceIndexed
  • ST_IndexedFacetDistance (the JTS name)

?

comment:3 by robe, 7 years ago

how about we go with the JTS name.

comment:4 by robe, 7 years ago

Owner: changed from pramsey to dbaston

Are you going to get to this in next week or so. If not can we push this to 2.5

comment:5 by robe, 7 years ago

Milestone: PostGIS 2.4.0PostGIS 2.5.0

pushing this to 2.5 since no word on it.

comment:6 by dbaston, 7 years ago

Priority: blockermedium

comment:7 by komzpa, 6 years ago

Milestone: PostGIS 2.5.0PostGIS 3.0.0

comment:8 by dbaston, 6 years ago

Resolution: wontfix
Status: newclosed

Now that Paul merged the distance tree code into liblwgeom, I see no reason to do this.

Note: See TracTickets for help on using tickets.