Error when applying Medial surface in Segment Editor hollow effect (with possible fix)

Development
1

I get an error when applying medial surface in hollow effect. The offending line is in SegmentEditorHollowEffect.py, line 157:

  kernelSizePixel = [int(kernelSizePixel[0],2), int(kernelSizePixel[1],2), int(kernelSizePixel[2],2)]

The int(…,2) specifies base 2, which doesn’t make sense.

I think this should read:

  kernelSizePixel = [int(kernelSizePixel[0]/2), int(kernelSizePixel[1]/2), int(kernelSizePixel[2]/2)]

I’m happy to submit a pull request if this looks correct.

Thanks,

-Hollister

2

Yes, please send a pull request. The change has been made by the automatic Python2->Python3 converter script.

3

Done. Thanks!

-Hollister

4

I don’t see a pull request here (checked it here https://github.com/Slicer/Slicer/pulls).