SimplifyPaths error

[philstopford]

I'm getting an error here and I don't see what the problem is:

TypeError: simplifyPaths(): incompatible function arguments. The following argument types are supported:
    1. (paths: List[List[Clipper2Lib::Point<__int64>]], epsilon: float, isOpenPath: bool = False) -> List[List[Clipper2Lib::Point<__int64>]]

Invoked with: [[[-250500, -500500], [-249500, -500500], [-249500, -249500], [-250500, -249500], [-250500, -500500]]], 0, False

It's a list of a list of integer points, so this should be legal, and the value of epsilon doesn't seem to influence things, and it's a closed path.

User error?

[paperluigis]

this seems to be a bug in the library as it should expect a numpy array for the paths, like with the pyclipr.ClipperOffset.addPaths method:

addPaths(self: pyclipr.pyclipr.ClipperOffset, path: List[numpy.ndarray[numpy.float64]], joinType: pyclipr.pyclipr.JoinType, endType: pyclipr.pyclipr.EndType = EndType.Polygon) -> None

[luktug-ltd]

@philstopford - the issue is that the scaleFactor for ClipperLib does not have a default argument. This will be amended in the next minor release to include a default (1e3) for the scaleFactor.

The function will accept both tuple/lists alongside numpy arrays.

import pyclipr

rect = [(0,0), (0,10), (10,10), (10,0), (0,0)]
rect2 = [(20,0), (20,10), (30,10), (30,0), (20,0)]

pyclipr.simplifyPath(rect, 0.001, 1000)
pyclipr.simplifyPaths([rect, rect2], 0.001, 1000)

It has revealed another issue that for simplification that the simplified paths are not scaled to their original coordinate frame. This will be fixed in the next release.