Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OperatorCut is returning incorrect result that has an extra piece. #80

Closed
alocke opened this issue Feb 20, 2015 · 3 comments
Closed

OperatorCut is returning incorrect result that has an extra piece. #80

alocke opened this issue Feb 20, 2015 · 3 comments
Assignees
Milestone

Comments

@alocke
Copy link
Member

alocke commented Feb 20, 2015

The input geometries look like this
inputgeometries
After the cut, Part 1 should look like this
correctpart1
But it looks like this
incorrectpart1

To reproduce, call OperatorCut with the geometries given below.

Polygon

{"rings":[[[-11604995.7656,8979981.3125],[-11367449.8649,8952923.2698],[-11417456.6674,8482935.2742],[-11604995.7656,8979981.3125]],[[-11413312.0818,8881319.661799997],[-11484157.3169,8872426.242700003],[-11434130.1311,8653413.995800003],[-11413312.0818,8881319.661799997]]],"spatialReference":{"wkid":102100,"latestWkid":3857}}

Polyline

{"paths":[[[-11671894.6618,9046978.563699998],[-11533321.8561,8971817.182099998],[-11367449.8649,8952923.2698],[-11594355.0515,8812292.009900004],[-11240774.4285,8776106.858400002]]],"spatialReference":{"wkid":102100,"latestWkid":3857}}
@stolstov
Copy link
Member

@alocke That's the output is obtained for the tolerance value of 1e-10. However 3857 should have the tolerance of 0.001, so this thin part should be deleted. A bug is in wkid to tolerance mapping.

@alocke
Copy link
Member Author

alocke commented Feb 20, 2015

Tested the fix in dev1.2.1. Works.

@stolstov stolstov added this to the 1.2.1 milestone Feb 20, 2015
@stolstov stolstov assigned alocke and unassigned stolstov Feb 26, 2015
@alocke
Copy link
Member Author

alocke commented Mar 12, 2015

Verified fixed in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants