MantisBT - VTK
View Issue Details
0015589VTK(No Category)public2015-07-17 09:242015-07-28 10:33
Cory Quammen 
David Gobbi 
lowminorhave not tried
closedfixed 
6.2.0 
6.3.0 
TBD
incorrect functionality
0015589: Failure case for vtkContourTriangulator
The attached contour causes the vtkContourTriangulator to fail. I have not looked into where the filter is failing, but post this here as a reference.

The filter succeeds on all other similar contours that I've tried (and even more complex contours). Just this one gives me problems.
No tags attached.
? ContourTriangulatorFail.vtp (11,854) 2015-07-17 09:24
https://www.vtk.org/Bug/file/9852/ContourTriangulatorFail.vtp
png Screen Shot 2015-07-17 at 9.55.11 AM.png (8,363) 2015-07-17 11:54
https://www.vtk.org/Bug/file/9853/Screen%20Shot%202015-07-17%20at%209.55.11%20AM.png
png
Issue History
2015-07-17 09:24Cory QuammenNew Issue
2015-07-17 09:24Cory QuammenAssigned To => David Gobbi
2015-07-17 09:24Cory QuammenFile Added: ContourTriangulatorFail.vtp
2015-07-17 11:54David GobbiFile Added: Screen Shot 2015-07-17 at 9.55.11 AM.png
2015-07-17 12:12David GobbiNote Added: 0034801
2015-07-20 15:12David GobbiNote Added: 0034810
2015-07-28 10:33Cory QuammenNote Added: 0034846
2015-07-28 10:33Cory QuammenStatusbacklog => closed
2015-07-28 10:33Cory QuammenResolutionopen => fixed
2015-07-28 10:33Cory QuammenFixed in Version => 6.3.0

Notes
(0034801)
David Gobbi   
2015-07-17 12:12   
Looks like a bug in the way the winding is computed. It's filling in all the concavities on the outside, rather than filling the interior. The problem doesn't appear to be degeneracy (vtkCleanPolyData didn't fix) or connectivity (vtkStripper didn't fix, even though it produced a single polyline as output).

The GetTriangulationError() returns "true" after execution, so the filter itself knows that something went wrong.
(0034810)
David Gobbi   
2015-07-20 15:12   
https://gitlab.kitware.com/vtk/vtk/merge_requests/439 [^]

The code in vtkPolygon.cxx for computing the polygon normal also looks wrong:
https://gitlab.kitware.com/vtk/vtk/blob/master/Common/DataModel/vtkPolygon.cxx#L192 [^]
What's worse, it has four copies of the polygon normal code, all with slight variations!
(0034846)
Cory Quammen   
2015-07-28 10:33   
David's patch was merged into master.