Changesets: Import 2017-06-07 23:51:09

Search Browse Back to Index

master 31eaf40b

2015-03-30 10:46:17

lassoan

Details Diff 		BUG: Updated OpenIGTLinkIF

Contains fixes for bugs in outgoing node sending.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24129 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild.cmake Diff File

master 2631d790

2015-03-27 18:05:21

jcfr

Details Diff 		ENH: Generalize Editor "propagationMode" to support arbitrary set of layers

This commit makes the Layer enum introduced in the r24106 public and uses
the Layer enum value to set the editor "propagationMode" value.

Reviewed-by: Andriy Fedorov <fedorov@bwh.harvard.edu>
Reviewed-by: Alireza Mehrtash <mehrtash@bwh.harvard.edu>
Reviewed-by: Steve Pieper <pieper@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24128 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Logic/vtkMRMLApplicationLogic.cxx Diff File
mod - Libs/MRML/Logic/vtkMRMLApplicationLogic.h Diff File
mod - Modules/Scripted/EditorLib/EditUtil.py Diff File
mod - Modules/Scripted/EditorLib/HelperBox.py Diff File

master 9976356d

2015-03-27 17:48:36

jcfr

Details Diff 		BUG: Update SliceViewAnnotation to avoid error when restoring scene view

This commit avoid the test "py_RSNA2012ProstateDemo" to produce the errors
reported below.


Notes:

(1) this commit does NOT fix the underlying issue. Indeed, after
restoring a scene view, no annotation are displayed.

(2) after restoring the "" scene view, there is still error related to
missing active camera:

WorldToView: no active camera, cannot compute world to view, returning 0,0,0

(3) after the test complete, when Slicer shutdown, the following warnings
are still reported:

Warning: In /path/to/VTKv6/Common/ExecutionModel/vtkAlgorithm.cxx, line 1421
vtkOpenGLImageMapper (0x18a48f50): Attempt to get connection index 0 for input port 0, which has 0 connections.




The associated error messages were:

// ------------
Traceback (most recent call last):
File "/path/to/Slicer-build/lib/Slicer-4.4/qt-scripted-modules/DataProbeLib/SliceViewAnnotations.py", line 622, in updateCornerAnnotations
self.makeAnnotationText(caller)
File "/path/to/Slicer-build/lib/Slicer-4.4/qt-scripted-modules/DataProbeLib/SliceViewAnnotations.py", line 808, in makeAnnotationText
sliceViewName = sliceNode.GetLayoutName()
AttributeError: 'NoneType' object has no attribute 'GetLayoutName'
// ------------

and

// ------------
Restoring scene view Preoperative T2w MRI ...
test_RSNA2012ProstateDemo (RSNA2012ProstateDemo.RSNA2012ProstateDemoTest) ... Traceback (most recent call last):
File "/path/to/Slicer-build/lib/Slicer-4.4/qt-scripted-modules/DataProbeLib/SliceViewAnnotations.py", line 605, in updateCornerAnnotations
self.makeAnnotationText(caller)
File "/path/to/Slicer-build/lib/Slicer-4.4/qt-scripted-modules/DataProbeLib/SliceViewAnnotations.py", line 808, in makeAnnotationText
foregroundOpacity = sliceCompositeNode.GetForegroundOpacity()
AttributeError: 'NoneType' object has no attribute 'GetForegroundOpacity'
// ------------

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24127 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/DataProbe/DataProbeLib/SliceViewAnnotations.py Diff File

master aeeed806

2015-03-27 17:00:17

lassoan

Details Diff 		ENH: Enabled output VTK error and warning testing for all qMRMLWidgetCxxTests

Now any qMRMLWidgetCxxTests test fails if VTK error or warning is logged during execution.

Several test errors are fixed (mainly due to missing or changed nodes) and where error/warning is expected exceptions added.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24126 3bd1e089-480b-0410-8dfb-8563597acbee
add - Libs/MRML/Core/Testing/L1.fcsv Diff File
mod - Libs/MRML/Core/Testing/diffusionEditor.mrml Diff File
mod - Libs/MRML/Core/vtkMRMLLayoutNode.h Diff File
mod - Libs/MRML/Widgets/Testing/CMakeLists.txt Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLLayoutManagerTest2.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLLayoutManagerTest3.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLLayoutManagerTest4.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLNavigationViewEventTranslatorPlayerTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLNavigationViewTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLNodeAttributeTableViewTest.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLNodeComboBoxTest3.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLSliceControllerWidgetTest.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLSliceWidgetTest2.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLThreeDWidgetEventTranslatorPlayerTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLThreeDWidgetTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLVolumeInfoWidgetEventTranslatorPlayerTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLVolumeThresholdWidgetEventTranslatorPlayerTest1.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLVolumeThresholdWidgetTest2.cxx Diff File
add - Libs/MRML/Widgets/Testing/qMRMLWidgetCxxTests.h Diff File
mod - Libs/vtkAddon/Testing/vtkLoggingMacrosTest1.cxx Diff File
mod - Libs/vtkAddon/vtkTestingOutputWindow.h Diff File

master 583caab9

2015-03-27 14:13:08

jcfr

Details Diff 		BUG: Fix missing DLL for extensions that use OpenMP (requires CMake 3.1)

This commit will ensure Plastimatch module bundled in SlicerRT extension
will work as expected.

Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

From: Greg Sharp <gregsharp.geo@yahoo.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24125 3bd1e089-480b-0410-8dfb-8563597acbee
mod - CMake/SlicerCPack.cmake Diff File

master 10cb2c13

2015-03-27 13:53:28

alexy

Details Diff 		ENH: reverted the order of adding scalars and normals so the scalars added first. Some modules take the first component.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24124 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/vtkTeem/vtkPolyDataTensorToColor.cxx Diff File

master bee786b7

2015-03-27 13:30:57

jcfr

Details Diff 		COMP: slicerFunctionAddPythonQtResources - Fix unknown argument warning

Use COPYONLY instead of COPY_ONLY.

This commit fixes the following warning (caught by CMake 3.2.1):

//-------------------------
CMake Warning (dev) at CMake/SlicerFunctionAddPythonQtResources.cmake:34 (configure_file):
configure_file called with unknown argument(s):

COPY_ONLY

Call Stack (most recent call first):
Modules/Scripted/ExtensionWizard/CMakeLists.txt:18 (slicerFunctionAddPythonQtResources)
//-------------------------

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24123 3bd1e089-480b-0410-8dfb-8563597acbee
mod - CMake/SlicerFunctionAddPythonQtResources.cmake Diff File

master e6bb7fc4

2015-03-27 10:33:02

jcfr

Details Diff 		COMP: Fix "unused-but-set-variable" warning in vtkMRMLNode

From: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24122 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/vtkMRMLNode.cxx Diff File

master 3e94490f

2015-03-27 10:32:59

jcfr

Details Diff 		ENH: Added support in vtkMRMLNode for multiple references to the same node

Fixed limitations of node references:

* Node reference modification request (SetAndObserveNthNodeReferenceID)
was ignored if only events were different. This caused an issue in
OpenIGTLinkIF, because different nodes in the same role required
different event observations and so the role-default events that were
restored from the scene were not correct.

* One node could only be referenced a single time only, otherwise the
most recently added/removed/modified reference overwrote all the
previous event observations. This limitation was not acceptable,
because even a temporarily created reference (the user selects a node
as input that is already selected as an input in another node selector)
could remove event observations. This caused an issue in Fiducial
registration wizard, which has two input fiducial lists, and when the
user selected one input fiducial list it often removed the observer
from the other input fiducial list.

Other related changes:

* Simplified node references implementation: all reference
add/remove/modification operations go through a single method
(SetAndObserveNthNodeReferenceID), previously there were many potential
execution path for these operations, which made it difficult to make
changes consistently.

* Fixed ObserverManager to update the observation even if only events or
priorities are changed.

* Made vtkMRMLNodeReference data members ReferencingNode, ReferencedNode,
Events accessible only by get/set functions to allow consistency checks.

* Renamed vtkMRMLNode::RemoveAllNodeReferenceIDs to RemoveNodeReferenceIDs
to be consistent with similar method names

From: Andras Lasso <lasso@queensu.ca>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24121 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/Testing/vtkMRMLNodeTest1.cxx Diff File
mod - Libs/MRML/Core/Testing/vtkMRMLTransformableNodeOnNodeReferenceAddTest.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLDisplayableNode.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLModelNode.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLNode.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLNode.h Diff File
mod - Libs/MRML/Core/vtkMRMLSelectionNode.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLTransformableNode.h Diff File
mod - Libs/MRML/Core/vtkMRMLVolumeNode.cxx Diff File
mod - Libs/MRML/Core/vtkObserverManager.cxx Diff File
mod - Libs/MRML/Core/vtkObserverManager.h Diff File

master a34a04bd

2015-03-27 03:22:07

jcfr

Details Diff 		BUG: Update CTK to include logger fixes

$ git shortlog 3d972114..657853b --no-merges
Andras Lasso (2):
BUG: Prevent infinite recursion in Qt log handler
BUG: Fixed heap corruption error by log handlers

Jean-Christophe Fillion-Robin (3):
Fix UseCTK.cmake to work for very simple project
Simplify ctkDICOMTableView constructor removing redundant call
Fix unused parameter warning in ctkLayoutViewFactory

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24120 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild/External_CTK.cmake Diff File

master 63cbdf31

2015-03-26 22:40:01

jcfr

Details Diff 		ENH: Lookup launcher and default settings files based on application name

Since following r24004 (ENH: Allow application name to be defined by
argv0) , we are sure the application name is set appropriately.
This commit will also ensure the application executable will lookup
launcher and default settings files based on the application name.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24119 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Base/QTCore/qSlicerCoreApplication.cxx Diff File
mod - CMake/SlicerMacroBuildApplication.cmake Diff File

master 22f85591

2015-03-26 19:30:21

jcfr

Details Diff 		ENH: Add support for default settings

If the file "<Slicer_HOME>/share/slicer-X.Y/SlicerDefaultSettings.ini"
exists, the corresponding key/value pairs are set into the user settings
if not already set.

Macro "slicerMacroBuildApplication" has also been updated to optionally
accept a "DEFAULT_SETTINGS_FILE" parameter.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24118 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Base/QTCore/qSlicerCoreApplication.cxx Diff File
mod - Base/QTCore/qSlicerCoreApplication.h Diff File
mod - Base/QTCore/qSlicerCoreApplication_p.h Diff File
mod - CMake/SlicerMacroBuildApplication.cmake Diff File

master 447189ae

2015-03-26 17:08:52

jcfr

Details Diff 		ENH: Support for hiding most of DICOM Browser controls through settings

This commit makes it possible to selectively hide most of the controls
associated with the DICOM browser. To hide all controls expect the "Import"
and "Load" button, the following entries should be added under
the "[DICOM]" group in the Slicer settings:

ActionRepair.visible=false
ActionExport.visible=false
ActionQuery.visible=false
ActionSend.visible=false
ActionRemove.visible=false
ActionViewMetadata.visible=false
AdvancedViewCheckBox.visible=false
HorizontalViewCheckBox.visible=false
BrowserPersistentCheckBox.visible=false
DatabaseButton.visible=false
TableDensityComboBox.visible=false

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24117 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/DICOMLib/DICOMWidgets.py Diff File

master a6a5b7ab

2015-03-26 16:17:28

lassoan

Details Diff 		BUG: Added back ITK image IO factory registration to qMRMLSliceWidgetTest2

It is required for successful loading of volume from nrrd file.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24116 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Widgets/Testing/qMRMLSliceWidgetTest2.cxx Diff File

master 2fd85453

2015-03-26 13:19:07

jcfr

Details Diff 		ENH: Extend slicer.util API introducing 'settingsValue' and 'toBool'

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24115 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Base/Python/slicer/util.py Diff File
mod - Modules/Scripted/DICOM/DICOM.py Diff File
mod - Modules/Scripted/DICOMLib/DICOMWidgets.py Diff File
mod - Modules/Scripted/DataProbe/DataProbeLib/SliceViewAnnotations.py Diff File

master fc04b75b

2015-03-26 11:30:57

lassoan

Details Diff 		BUG: Fixed more silsntly failing tests

qMRMLSliceWidgetTest2
qMRMLVolumeThresholdWidgetTest2


git-svn-id: http://svn.slicer.org/Slicer4/trunk@24114 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/Testing/vol.mrml Diff File
mod - Libs/MRML/Core/Testing/vol_and_cube.mrml Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLSliceWidgetTest2.cxx Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLVolumeThresholdWidgetTest2.cxx Diff File

master 15ddc665

2015-03-26 10:39:42

lassoan

Details Diff 		BUG: Log warnings instead of throw exception from vtkITKArchetypeImageSeriesReader

If an exception is thrown in the middle of a pipeline update then the state of the pipeline becomes inconsistent,
therefore all exceptions should be caught, logged as error, and return with an error status.

Removed commented out macros that log message instead of throwing exception, because there is no automatic way
for converting error handling by exception to error handling by logging&returning with error status.
Exceptions that are thrown in the VTK file are removed and exceptions that are thrown from other classes are caught.

Also changed catch(...) to catch ITK exceptions instead. Catch(...) might be only considered in a high-level application error handler (but even then the application state may be irrecoverable), but it must not be caught ever in library code.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24113 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/vtkMRMLVolumeArchetypeStorageNode.cxx Diff File
mod - Libs/vtkITK/vtkITKArchetypeImageSeriesReader.cxx Diff File

master 43217535

2015-03-26 00:30:01

lassoan

Details Diff 		BUG: Fixed silently failing qSlicerWidgetTest2

qSlicerWidgetTest2 had two important errors:


1. The test actually failed

It could not load the input image, as reported on the dashboard:

http://slicer.cdash.org/testDetails.php?test=6130889&build=403273

ERROR: In ..\..\..\..\..\Slicer-1\Libs\MRML\Core\vtkMRMLVolumeArchetypeStorageNode.cxx, line 367
vtkMRMLVolumeArchetypeStorageNode (0000000004D4BE20): ReadData: Cannot read file as a volume of type Volume[fullName = C:/D/N/Slicer-1-build/Slicer-build/Base/QTGUI/Testing/Cxx/share/MRML/Testing/TestData/fixed.nrrd]
Number of files listed in the node = 0.
File reader says it was able to read 0 files.
File reader used the archetype file name of C:/D/N/Slicer-1-build/Slicer-build/Base/QTGUI/Testing/Cxx/share/MRML/Testing/TestData/fixed.nrrd []

The problem was that no ITK image IOs were registered.
Fixed by adding ITKFactoryRegistration to the test.
Also added proper logging of ITK exception data in vtkMRMLVolumeArchetypeStorageNode.


2. The test did not report the failure

The test just logged the error but nobody noticed (it was not reported as an error on the dashboard).

Fixed by using VTK output testing by adding these 3 lines of code:

#include <vtkTestingOutputWindow.h>
...
TESTING_OUTPUT_INIT;
...
TESTING_OUTPUT_ASSERT_WARNINGS_ERRORS(0);

Now if there is any warnings or error during the test then it returns with failure.

VTK output testing should be similarly added for more (probably all) tests.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24112 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/vtkMRMLVolumeArchetypeStorageNode.cxx Diff File
mod - Libs/MRML/Widgets/Testing/CMakeLists.txt Diff File
mod - Libs/MRML/Widgets/Testing/qMRMLSliceWidgetTest2.cxx Diff File

master 74807e1b

2015-03-26 00:20:09

lassoan

Details Diff 		ENH: Added VTK output window class for capturing VTK errors/warnings during testing

Currently VTK errors and warnings are ignored in automatic test, while they often
detect real errors. This class helps detecting unexpected errors and warnings.

Example use:

// Initialize output window. All messages are logged to the console.
TESTING_OUTPUT_INIT;

...

// Set up a checkpoint, we restart counting messages from this point
TESTING_OUTPUT_RESET;
...do something that should not log error messages
// Exit with EXIT_FAILURE if any warnings or errors are logged
TESTING_OUTPUT_ASSERT_WARNINGS_ERRORS(0);

TESTING_OUTPUT_RESET;
...do something that should log error messages
TESTING_OUTPUT_ASSERT_ERRORS_MINIMUM(1);


git-svn-id: http://svn.slicer.org/Slicer4/trunk@24111 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/vtkAddon/CMakeLists.txt Diff File
mod - Libs/vtkAddon/Testing/vtkLoggingMacrosTest1.cxx Diff File
add - Libs/vtkAddon/vtkTestingOutputWindow.cxx Diff File
add - Libs/vtkAddon/vtkTestingOutputWindow.h Diff File

master d354fef4

2015-03-25 23:30:40

jcfr

Details Diff 		COMP: Update VTKv6 to include fix for vs2013 compilation

//--------------
commit c501ccc5df46d33fa15ae1e5df94459f3b98c52b
Author: Isaiah Norton <isaiah.norton@gmail.com>
Date: Wed Mar 25 22:51:37 2015 -0400

Visual Studio 2013 requires the <algorithm> header.
//--------------

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24110 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild/External_VTKv6.cmake Diff File

master 34bff896

2015-03-25 18:17:51

jcfr

Details Diff 		BUG: Update CTK to include DICOMTableView fixes

$ git shortlog a358caf..3d97211 --no-merges
Andreas Fetzer (2):
Fixed duplicate connection to Qt signal/slots in ctkDICOMTableView
init now calls setDicomDatabase, which then calls setUpTableView

Jan Zieher (1):
getter for XNAT sessionId

Jean-Christophe Fillion-Robin (1):
DICOMTableView: Do not clear selection if user click outside of existing rows

Sascha Zelzer (2):
Fixed missing include dirs for plug-in test drivers.
Don't treat CTKData as a CMake package. Fixes a small issue in 0000532.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24109 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild/External_CTK.cmake Diff File

master af5256dd

2015-03-25 04:37:20

jcfr

Details Diff 		BUG: Fixed editor behaviour in restoring correct label.

To reproduce the issue:
* Load a volume, go to editor and create a merge labelmap.
* With Paint effect paint something with label 1.
* While on Label 1 select eraser, erase something and go back to painteffect.
* Select another label e.g 2.
* Paint something with painteffect. It works fine (paint with label 2).
* Choose another effect (e.g draw or want effect) and it would go back to the label 1 as it only stores the backup label when eraser is selected.

Fix: Backup label should also be set when a new color (not zero) is chosen not only after switching to eraser.

From: Alireza Mehrtash <mehrtash@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24108 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/EditorLib/EditUtil.py Diff File

master 970696ad

2015-03-25 03:59:09

jcfr

Details Diff 		ENH: Update SimpleITK to v0.9b01 with few patches

This update includes:

d5b32be Merge topic 'SIMPLEITK-457_AddExhaustiveOptimizer'
2e4af8c Merge topic 'SIMPLEITK-622_LevelSetMotionFixImageNames'
a15c3f2 Merge topic 'SIMPLEITK-623_AddOptmizerWeights'

From: Bradley Lowekamp <blowekamp@mail.nih.gov>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24107 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild/External_SimpleITK.cmake Diff File

master bb9b6623

2015-03-25 03:27:35

jcfr

Details Diff 		ENH: Add option to control the PropagateVolumeSelection behavior

* Options added so volume selection can propagate only to background,
foreground and label layers.

* HelperBox and EditUtil were modified to use the new propagation mode.

Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

From: Alireza Mehrtash <mehrtash@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24106 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Logic/vtkMRMLApplicationLogic.cxx Diff File
mod - Libs/MRML/Logic/vtkMRMLApplicationLogic.h Diff File
mod - Modules/Scripted/EditorLib/EditUtil.py Diff File
mod - Modules/Scripted/EditorLib/HelperBox.py Diff File

master 549cfeb0

2015-03-25 02:00:59

jcfr

Details Diff 		BUG: Fix py_SubjectHierarchyGenericSelfTest

This commit re-adds function "onDatabaseDirectoryChanged" previously
removed by r24104 (ENH: Refactor DICOM module to allow DICOMDetailsPopup
standalone instantiation) and fixes the failing test.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@24105 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Scripted/DICOM/DICOM.py Diff File
 First  Prev  1 2 3 ... 10 ... 20 ... 30 ... 40 ... 50 ... 60 ... 70 ... 75 76 77 78 79 80 81 ... 90 ... 100 ... 110 ... 120 ... 130 ... 133 134 135  Next  Last