Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#4215 closed defect (fixed)

Drop usage of GDALPolygonize in DT_DumpAsPolygons

Reported by: komzpa Owned by: komzpa
Priority: medium Milestone: PostGIS 2.4.6
Component: raster Version: 2.4.x
Keywords: Cc:

Description (last modified by komzpa)

http://brian-mcgill-4.ums.maine.edu/postgis_zonal.pdf :

However this function is
inadequate for the current purpose – it merges pixels with the same value into larger polygons. In
principle this should still be acceptable but the actual implementation (in a poorly documented
“feature”) does this by converting every value to an integer. So pixels with a temperature of
13.49 and 12.51 are merged incorrectly (and the value stored is 13 which is also incorrect) 

Looking at code we have branching on whether GDAL>F<Polygonize is available, and that looks hardcoded to "no". We're supporting GDAL 1.10+ and GDALFpolygonize is available since 1.9.0, so branching can be removed.

Change History (5)

comment:1 by komzpa, 6 years ago

Description: modified (diff)

comment:2 by komzpa, 6 years ago

Owner: changed from dustymugs to komzpa
Status: newassigned

comment:3 by komzpa, 6 years ago

Resolution: fixed
Status: assignedclosed

In 16946:

[raster] Use GDALFPolygonize unconditionally

GDAL 1.10 is already required and 1.9 contains it, so no check needed.

Closes #4215

comment:4 by komzpa, 6 years ago

In 16947:

[raster] Use GDALFPolygonize unconditionally

GDAL 1.10 is already required and 1.9 contains it, so no check needed.

Closes #4215

comment:5 by komzpa, 6 years ago

In 16948:

[raster] Use GDALFPolygonize unconditionally

GDAL 1.10 is already required and 1.9 contains it, so no check needed.

Closes #4215

Note: See TracTickets for help on using tickets.