Skip to content

STYLE: Remove dead code for BlobDetection and ConnectedComponent CLI modules

Sam Horvath requested to merge remove-not-built-clis into master

Created by: jcfr

The code has NOT been compiled since r20744 [1] from August 2010.

[1] http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20744)

Associated issue was: https://issues.slicer.org/view.php?id=2809)

And a excerpt from @pieper reply from 2012 on the slicer-devel [2] mailing list thread entitled Re: Adding new modules to Slicer core? :

I haven't looked at this in detail, but I think the module has the potential for wide utility, but since it's currently disabled it's not yet widely used. It also seems that it doesn't have documentation, which is probably why it is currently disabled. So yes, for now I think BlobDetection and the related module ConnectedComponent should move to an extension (as should a few other specialized modules). We should probably make this a group project at some point, but as we discussed on yesterday's call performance optimization and ITKv4 are higher priorities right now :)

[2] http://slicer-devel-archive.65872.n3.nabble.com/Adding-new-modules-to-Slicer-core-tp4025907p4026923.html

For reference, see also associated discourse topic: https://discourse.slicer.org/t/slicer-5-0-deprecation-discussion-wiki/2377/8?u=jcfr

Merge request reports