Difference boolean operation (e.g. A-B) leads to crash

[mauigna06]

Hi @zippy84

Thanks for the amazing software

I can replicate crashes with the file below

vtk_bool_difference_A-B_operation_bug.zip

Hope you can help

PS: I worked around it by adding some filters to the pipeline that creates the cylinders

[zippy84]

Hello,

do you use the newest version of my lib?

Here is what I get:

The code I used to created it is:

#!/usr/bin/env python
# *-* coding: UTF-8 *-*

import sys
sys.path.extend(['/home/zippy/vtkbool/build/lib/python3.11/site-packages/vtkbool'])

from vtkmodules.vtkIOLegacy import vtkPolyDataReader, vtkPolyDataWriter
from vtkmodules.vtkIOGeometry import vtkSTLReader, vtkSTLWriter
from vtkmodules.vtkFiltersCore import vtkCleanPolyData, vtkPolyDataNormals, vtkTriangleFilter
from vtkBool import vtkPolyDataBooleanFilter

readerA = vtkPolyDataReader()
readerA.SetFileName('A.vtk')

readerB = vtkPolyDataReader()
readerB.SetFileName('B.vtk')

bf = vtkPolyDataBooleanFilter()
bf.SetInputConnection(0, readerA.GetOutputPort())
bf.SetInputConnection(1, readerB.GetOutputPort())
bf.SetOperModeToDifference2()

clean = vtkCleanPolyData()
clean.SetInputConnection(bf.GetOutputPort())

writer = vtkPolyDataWriter()
writer.SetFileName('result.vtk')
writer.SetInputConnection(clean.GetOutputPort())
writer.Update()

[zippy84]

Respectively with SetOperModeToDifference:

[mauigna06]

do you use the newest version of my lib?

I don't know how to check, I think the files of vtkbool are inside this folder:
https://github.com/PerkLab/SlicerSandbox/tree/master/CombineModels/Logic

It appears they are a few years old

[zippy84]

Yes, the files there are really old. You should upgrade to version 3.0. The new version is much better!! 😃

[BigBrotherHui]

Yes, the files there are really old. You should upgrade to version 3.0. The new version is much better!! 😃

where is the newest version of vtkbool?the slicersandbox have't update the logic 3 years age

[lassoan]

Thank you, we've updated vtkbool in the Slicer extension now. The current vtkbool version is indeed much cleaner, properly wrapped, and available as a VTK module, so in the future we'll not have to vendorize it anymore.

The only changes I had to make in vtkbool to allow it to build on Windows were:

• Add #define _USE_MATH_DEFINES to Utilities.cxx (without that build failed due to M_PI symbol was not found).
• Exclude the _Pair class from wrapping. Maybe it was due to something not being optimal in our CMake files, as Utilities.cxx/.h should not have been even considered by the wrapper; but since Utilities.h was included in Python-wrapped header files, I did not see any easy way around it. For a nice and clean interface, all the private implementation details (such as StripPt, StripPtR, PStrips, etc. classes, and #include "Utilities.h") could be removed from vtkPolyDataContactFilter.h and vtkPolyDataBooleanFilter.h header files (and these classes would be included only in the .cxx files).

[zippy84]

• Add #define _USE_MATH_DEFINES to Utilities.cxx (without that build failed due to M_PI symbol was not found).

This is defined by my CMakeLists.txt 😄