search

MedicalImageAnalysisTutorials / SlicerCochlea

Multi-modal cochlea Images registration, segmentation and analysis
13 stars 4 forks source link

ModuleNotFoundError: No module named 'Elastix' #16

Closed jamesobutler closed 5 years ago

jamesobutler commented 5 years ago

There are failing tests for the Slicer preview because of an error importing Elastix.

SlicerCochlea is currently not specifying any dependencies when it appears it should be specifying SlicerElastix. The code that needs to be updated is here for both the “master” and “4.10” branch of the ExtensionsIndex repo.

See here how the SequenceRegistration extension specified SlicerElastix as a dependency.

idhamari commented 5 years ago

Thanks for your comment. I think the problem comes from not updating the extension index. I just sent a new pull request. There is a new VisSimCommon module that depends on Ealstix.

jamesobutler commented 5 years ago

With the PR, the failing test is still happening. This issue should remain open.

I have asked a question about the issue which can be viewed at https://discourse.slicer.org/t/failing-tests-importerror-no-module-named-elastix/7202

idhamari commented 5 years ago

Thanks for your effort. All branches are updated! I hope the next build has no failing tests.

jcfr commented 5 years ago

Since this is issue has been fixed by https://github.com/lassoan/SlicerElastix/pull/18, I suggest to close this issue.

idhamari commented 5 years ago

done!