Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#4131 closed defect (fixed)

ConcaveHull sfcgal test lost

Reported by: komzpa Owned by: robe
Priority: medium Milestone: PostGIS 2.5.0
Component: postgis Version: master
Keywords: Cc:

Description

In https://github.com/postgis/postgis/commit/5a6b8a512fd9c72402520a167571276e4504852f a switch to sfcgal backend got lost.

Switch should be returned into its place, and probably \i concave_hull.sql.

After that probably https://trac.osgeo.org/postgis/ticket/4126 will show up once again.

Change History (4)

comment:1 by robe, 6 years ago

Owner: changed from pramsey to robe

comment:2 by robe, 6 years ago

Just a note on my local after turning on sfcgal, still waiting for concave hull test to finish.

Almost tempted to turn off sfcgal inside the function :)

comment:3 by robe, 6 years ago

Resolution: fixed
Status: newclosed

In 16659:

put back old sfcgal test (except use \i, and have the query be what was in sfcgal one), the sfcgal one was different
Closes #4131
Revise ST_ConcaveHull to use ST_UnaryUnion(ST_Collect..) (cause I know sfcgal doesn't replace with it's own implementation / a bit of a cheat)
Take out some ST_Union calls I think are over compensating.
References #3638

comment:4 by robe, 6 years ago

Forgot to mention I left in Komzpa's new crueler (realer) test for ST_ConcaveHull, but not bothering to have it run in SFCGAL (it might run reasonably now), but I called it

concavehull_hard

Note: See TracTickets for help on using tickets.