vnarayan13 / slicer-opencad Goto Github PK

Currently on version 4.6 the Segment CAD extension doesn't run because there are incorrect definitions of single-element Qt string in SegmentCAD.py and there is a missing pair of () in SegmentCADLogic.py

We are planning to remove Slicer core modules that have been superseded by other modules.
https://discourse.slicer.org/t/removing-legacy-modules-in-slicer5/11371

We have noticed that this extension currently uses legacy charts infrastructure (Plots, and Tables modules).

If you want your extension to be available for the upcoming Slicer5 release, please remove dependency to deprecated modules. If you have any questions or need any help with the upgrade, you can write here or post to the Slicer forum. Thank you for your contribution.

Hi,
It is really nice to use the module you created! I just have a question about the "Compactness 1" in "Heterogeneity CAD". According to the help page, it is supposed to be the ratio of volume to the (surface area)^(1.5). But as i look into the code in HeterogeneityCAD/FeatureExtractionLib/MorphologyStatistics.py 000

def compactness1 (self, surfaceArea, volumeMM3):
return ( (volumeMM3) / ((surfaceArea)**(2/3.0) * math.sqrt(math.pi)) )

I wonder if it should be (surfaceArea)**(3/2.0) instead.
Thanks for your time!

Hi,
I just have another question about the "Spherical disproportion" in "Heterogeneity CAD". According to the help page, it is supposed to be ratio of "the surface area of the image ROI" to "the surface area of a sphere with the same volume as the image ROI". But as i look into the code in HeterogeneityCAD/FeatureExtractionLib/MorphologyStatistics.py

def sphericalDisproportion (self, surfaceArea, volumeMM3):
R = ( ( 0.75 * (volumeMM3) ) / (math.pi)**(1/3.0) )
return ( (surfaceArea)/(4*math.pi*(R**2)) )

The volume of a sphere with radius "R" is (4/3) * pi * R^3.
In getting the "R" above, I wonder if it should be ( ( 0.75 * (volumeMM3) / (math.pi) )**(1/3.0) ) instead.
Thanks for your time!