AIM-Harvard / SlicerRadiomics

A Slicer extension to provide a GUI around pyradiomics
BSD 3-Clause "New" or "Revised" License
106 stars 48 forks source link

CDash test failed #41

Closed JoostJM closed 6 years ago

JoostJM commented 6 years ago

@jcfr, it appears the generic module test for slicer radiomics failed. It is complaining about a call to VC with python arguments.

test_widgetRepresentation (qSlicerSlicerRadiomicsModuleGenericTest.qSlicerSlicerRadiomicsModuleGenericTest) ... Traceback (most recent call last):
  File "C:/D/N/E-0/Radiomics-build/inner-build/lib/Slicer-4.9/qt-scripted-modules/SlicerRadiomics.py", line 61, in setup
    self._addCustomizationSection()
  File "C:/D/N/E-0/Radiomics-build/inner-build/lib/Slicer-4.9/qt-scripted-modules/SlicerRadiomics.py", line 295, in _addCustomizationSection
    self.parameterFilePathLineEdit = ctk.ctkPathLineEdit(filters=ctk.ctkPathLineEdit.Files)
ValueError: Calling C++ functions with Python keywords is not supported! Function: ctkPathLineEdit(QWidget parent) Keywords: {'filters': 2}
ok

http://slicer.cdash.org/testDetails.php?test=8615594&build=1191518

Moreove, the unit-tests on windows also appear to be failing, but I don't see any output. The Linux/Mac tests pass, but they are tested using an earlier commit.

JoostJM commented 6 years ago

Update: I ran the unit test locally on my Linux build, no errors.

JoostJM commented 6 years ago

Update 2: The error in the generic module test appears to be due to the transition to Qt5/VTK9, I fixed this in f158e5b.

JoostJM commented 6 years ago

@Radiomics/developers Some updates on the testing cdash: