Opened 5 years ago

Last modified 21 months ago

#4680 new enhancement

Implement ST_GetFaceGeometry by following rings, not face labels

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS Fund Me
Component: topology Version: master
Keywords: Cc:

Description

At the moment ST_GetFaceGeometry builds an area from the edges having the target face known to be on one of their sides. This requires re-determining edge connection when such information is already available (and used by GetRingEdges).

This ticket is to enhance the function to use edge linking instead, as it should be faster (no need to use the GEOS polygonizer) and more robust (would still work in case face labels are invalid).

Eventually, we could have a switch to decide whether to use face labels or edge linking.

Of course face label would still be needed to find at least one face edge for each face ring…

Change History (5)

comment:1 by Algunenano, 4 years ago

Milestone: PostGIS Fund Me

comment:2 by strk, 4 years ago

Important info: this would NOT be easy to make faster than GEOS with only using SQL, as the already complex recusrive CTE used to find a ring of edges would need to be iterated to find *all* rings, and for faces with a lot of holes this would be very very expensive, unless it can be done with a single SQL statement

comment:3 by strk, 3 years ago

For the record: we do have some C code to build polygons under the lwt_Polygonize function, we could do a single database access to fetch all edges advertising the given face on any of their sides and do the graph walking in memory. Note that the direct edge linking is not always enough as it would build Maximal Edge Rings while we'd need a Minimal Edge Rings to avoid self-intersecting shells invalidities (and dangling pointers).

comment:4 by strk, 3 years ago

Reimplementing ST_GetFaceGeometry would help with performance issues like #4966

comment:5 by strk, 21 months ago

One component of a reimplementation of GetFaceGeometry would be building the shell part, which is the focus of this merge request: https://gitlab.com/postgis/postgis/-/merge_requests/30

Note: See TracTickets for help on using tickets.