SlicerRT not compatible with nightly

bug
(Alex Vergara) #1

When installing SlicerRT it says

File “/Users/alex/Applications/Slicer.app/Contents/Extensions-28138/SlicerRT/lib/Slicer-4.11/qt-scripted-modules/BatchStructureSetConversion.py”, line 233
except Exception, e:
^
SyntaxError: invalid syntax

0 Likes

(Csaba Pinter) #2

Thanks for the report! We are in transition to Python3, which is not a trivial task, so it might take a while until everything is functional again.

I’ll fix this asap though. And please keep reporting anything new that comes up!

0 Likes

(Jean Christophe Fillion Robin) #3

@cpinter I will submit a PR on SlicerRT with a first round of updates.

0 Likes

(Alex Vergara) #4

Traceback (most recent call last):
File “/Users/alex/Applications/Slicer.app/Contents/lib/Slicer-4.11/qt-scripted-modules/DICOMLib/DICOMWidgets.py”, line 730, in getLoadablesFromFileLists
loadablesByPlugin[plugin] = plugin.examineForImport(fileLists)
File “/Users/alex/Applications/Slicer.app/Contents/Extensions-28138/SlicerRT/lib/Slicer-4.11/qt-scripted-modules/DicomRtImportExportPlugin.py”, line 43, in examineForImport
for loadableIndex in xrange(0,loadablesCollection.GetNumberOfItems()):
NameError: name ‘xrange’ is not defined
Warning: Plugin failed: DicomRtImportExportPlugin

See python console for error message.
DICOM Plugin failed: name ‘xrange’ is not defined

0 Likes

(Csaba Pinter) #5

@jcfr I’m working on it right now

@Alex_Vergara Please stand by with more error reports until I have committed the first fixes

0 Likes

(Alex Vergara) #6

the last error was not on loading but when trying to import a DICOMRT

0 Likes

(Jean Christophe Fillion Robin) #7

@cpinter and I just synced offline and Csaba will continue his effort and i will help as needed later.

0 Likes

(Csaba Pinter) #8

@Alex_Vergara SlicerRT works in the latest nightly. There is one more error coming from Slicer core when you load an RTImage, but otherwise it seems to works well. I’ll still do some tests to make sure, but I think the main functions should be fine now. Thanks for the report!

1 Like