[GEOS] Add simplify function to php/mapscript

[jlacroix]

There's a simplify function in GEOS and it would be cool to have access to it through mapscript. See the following exchange to see the interest around it.

 http://www.nabble.com/vertex-simplification-td13442613.html

[jlacroix]

patch to include the simplify functions in php/mapscript

[dmorissette]

To summarize, the patch adds the following functions in mapgeos.c:

shapeObj *msGEOSSimplify(shapeObj *shape, double tolerance);
shapeObj *msGEOSTopologyPreservingSimplify(shapeObj *shape, double tolerance);

... and the following methods in MapScript?:

shapeObj shapeObj.simplify(double tolerance)
shapeObj shapeObj.topologyPreservingSimplify(double tolerance)

Steve, if you agree with this addition then we'll commit the patch for PHP MapScript? and you can then add the methods to SWIG MapScript?.

[sdlime]

Sounds ok to me. Just assign to me once you complete the PHP part and then I'll do SWIG and then assign to the documentation component.

Steve

[dmorissette]

Cool. I'll let Julien take care of it and he will reassign to you after.

[sdlime]

Do you guys want to get this into 5.6 yet? Only addition would be adding methods on the Swig side. I'd do that if you want to move forward.

Steve

[dmorissette]

Too late for 5.6, but let's take care of this early in 6.0

[sdlime]

Patch won't apply clean against trunk, but it was just the ZEND_DEBUG hunk which looks not to be a problem. I also added methods to shape.i in the Swig stuff. I tested the compile there but not for PHP. Could someone verify the PHP build please?

Moving to documentation component...

Steve

[dmorissette]

Assigned to Jeff for docs. The new methods to document are:

shapeObj shapeObj.simplify(double tolerance)
shapeObj shapeObj.topologyPreservingSimplify(double tolerance)

[dmorissette]

BTW, the changeset for comment:7 was r9558.

[dmorissette]

I found out at the last minute while packaging 5.6.0-rc1 that this addition broke the buildbots which are using GEOS 2.2.3. It seems that GEOSSimplify and GEOSTopologyPreserveSimplify are only in GEOS 3.0+.

I committed a patch in r9569 that enables the new methods only for GEOS 3.0+. This is included in 5.6.0-rc1.

I also created ticket #3215 about possibly bumping up the GEOS requirement to 3.0+ in Mapserver 6.0.

[havatv]

Fixed for 6.0 and trunk documentation in r12034 and r12035 (a geos version error had to be corrected...).