Model Registration Module Error

sfglio · June 17, 2021, 6:33pm

How can I fix the following error message?

Traceback (most recent call last):
File “/Applications/Slicer.app/Contents/Extensions-29959/SlicerIGT/lib/Slicer-4.13/qt-scripted-modules/ModelRegistration.py”, line 167, in onApplyButton
logic.run(inputSourceModel, inputTargetModel, outputSourceToTargetTransform, self.typeSelector.currentIndex, self.iterationSpin.value )
File “/Applications/Slicer.app/Contents/Extensions-29959/SlicerIGT/lib/Slicer-4.13/qt-scripted-modules/ModelRegistration.py”, line 185, in run
self.delayDisplay(‘Running iterative closest point registration’)
AttributeError: ‘ModelRegistrationLogic’ object has no attribute ‘delayDisplay’

I am using the latest nighty version of slicer…

Kind regards

jamesobutler · June 17, 2021, 9:24pm

delayDisplay in the ScriptedModuleLogic class had been deprecated due to delayDisplay being for testing purposes and had been in the ScriptedModuleTest class in addition to being available from anywhere using slicer.util.delayDisplay. The deprecation of delayDisplay in the Logic class was enforced as it was removed in favor of only having the other locations. This was ENH: Move testing methods in scripted logic class to test class · Slicer/Slicer@71f236b · GitHub.

Therefore the ModelRegistration module needs to be updated here:

github.com

SlicerIGT/SlicerIGT/blob/5cb6a0eda5a38cf4f4d04c475e9cbca7227ad200/ModelRegistration/ModelRegistration.py#L185

    
      
          class ModelRegistrationLogic(ScriptedLoadableModuleLogic):
            """This class should implement all the actual
            computation done by your module.  The interface
            should be such that other python code can import
            this class and make use of the functionality without
            requiring an instance of the Widget
            """
          
          
  def run(self, inputSourceModel, inputTargetModel, outputSourceToTargetTransform, transformType=0, numIterations=100 ):
          
          
    self.delayDisplay('Running iterative closest point registration')
          
          
    icpTransform = vtk.vtkIterativeClosestPointTransform()
              icpTransform.SetSource( inputSourceModel.GetPolyData() )
              icpTransform.SetTarget( inputTargetModel.GetPolyData() )
              icpTransform.GetLandmarkTransform().SetModeToRigidBody()
              if transformType == 1:
                icpTransform.GetLandmarkTransform().SetModeToSimilarity()
              if transformType == 2:
                icpTransform.GetLandmarkTransform().SetModeToAffine()
              icpTransform.SetMaximumNumberOfIterations( numIterations )

sfglio · June 18, 2021, 6:58pm

Thank you for your reply. But what exactly needs to be updated in L185 asking as a python newbie? Is that something I can fix?

In my script it looks exactly as indicated above.

jamesobutler · June 18, 2021, 7:18pm

You could change it to

slicer.util.delayDisplay('Running iterative closest point registration')

If you’re able to contribute this change by submitting a PR to GitHub - SlicerIGT/SlicerIGT: Modules supporting image-guided interventions in 3D Slicer. that would be helpful! Then other people that download the extension won’t experience the same issue you have found.

sfglio · June 18, 2021, 8:43pm

Yes this works. I tried to open a PR in GitHub - SlicerIGT/SlicerIGT: Modules supporting image-guided interventions in 3D Slicer.
[/quote] however I only managed to open an issue there…sorry

Topic		Replies	Views
Error running Elastix Community	1	155	January 11, 2024
Distance Based Registration in SlicerProstate Support	1	168	September 28, 2021
Problem with General Registration (Elastix) Support registration , segmentation	4	511	September 9, 2023
Sequence registration module Support	5	777	August 22, 2023
ROI Registration Non-Functional SlicerCMF registration	0	363	November 9, 2020

Model Registration Module Error

Related topics