[vtk-developers] New class: vtkGraphLargestConnectedComponent

Jeff Baumes jeff.baumes at kitware.com
Fri May 21 11:57:18 EDT 2010


> Any other reviews?

I'll add one: You shouldn't directly connect an input's producer port
to an internal algorithm like you are doing with
vtkBoostConnectedComponents. Instead you should make a new instance of
the graph and perform a ShallowCopy(), then send that into the
internal algorithm. Similarly, you should ShallowCopy() the output of
the last filter into the real filter output object.

Also, we have been putting the Boost prefix on anything that depends
on Boost. The name you have is already remarkably long, but perhaps
something like vtkBoostExtractLargestComponent would work?

Jeff



More information about the vtk-developers mailing list