COMP: Fix unused variable warnings

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21105 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix unused variable warnings in qMRMLVolumeWidget

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21106 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Return appropriate IDs in vtkMRMLPETProceduralColorNode::GetTypeAsIDString()

From Julien: While this will fix the problem, it doesn't fix the root
cause of the issue.
a) there is a duplication of information between "IDs" and "type strings"
b) I don't think a MRML node should know about how IDs are set.
It doesn't even set the SingletonTag to itself, however
vtkMRMLColorLogic is the one that sets the ID (by setting the singleton),
so vtkMRMLColorLogic should be the one that know about IDs. Actually,
to be really consistent, only the scene knows about IDs
(vtkMRMLScene::GenerateUniqueID).


See http://www.na-mic.org/Bug/view.php?id=2613

Fixes #2613

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21107 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: change debugging print out to a debug macro

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21108 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: volume rendering initialization procedure has been updated so that no extra ROIs should be created when switching between steps; fixes #2598
BUG: popup error message display added when registration failed, addresses #2587; the root of the issue is #1635, which remains unresolved, but causes problems only on some Linux configurations


git-svn-id: http://svn.slicer.org/Slicer4/trunk@21109 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: don't print out loading module debugging statement during testing as it can cause dashboard output to be truncated before the extension test output is shown

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21110 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Remove vtkMRMLXXXColorYYYNode::GetTypeAsIDString()

a) there was a duplication of information between "IDs" and "GetTypeAsString"
b) A color node should not know about how IDs are set.
It doesn't even set the SingletonTag to itself,
however vtkMRMLColorLogic is the one that sets the ID
(by setting the singleton), so vtkMRMLColorLogic should be the one
that knows about IDs.
Actually, to be really consistent, only the scene knows about IDs
(see vtkMRMLScene::GenerateUniqueID).

Closes #2613

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21111 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix CDash errors/warnings reporting for unix parallel build

Setting the variable CTEST_USE_LAUNCHERS_DEFAULT in the environment
to enable CTest launcher reliably will be supported by default in either
CMake 2.8.10 or 2.8.11.

In the mean time, your local installation of CMake can be modified based on
the following topic: https://github.com/jcfr/CMake/tree/fix-ctest-use-launchers-for-superbuild

See http://na-mic.org/Mantis/view.php?id=2468

Fixes #2468

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21112 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Extend vtkMRMLDisplayNode documentation

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21113 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Add center view button in 3D view controller toolbar

Issue #2297

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21114 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: flag the Loading and Uninstantiating module print outs via the verbose module discovery flag rather than the testing one

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21115 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Both CTest and CMake should have CTEST_USE_LAUNCHERS set

See http://na-mic.org/Mantis/view.php?id=2468

Fixes #2468

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21116 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: testing the Reporting module found that another extension had registered a node with the same tag, so changing a debug to a warning, and also removing the double register of the scalar volume node

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21117 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: handle case where no dicom database exists for RSNA dicom test

Handles the case of a fresh install where the dicom module
has never been run before.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21118 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: free the indexer instance so that indexing threads will finish

The indexer uses the QFuture mechanism, so the operation may not have
completed when the call returns and the next step begins.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21119 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: better managment of startup conditions for dicom database

In particular for self testing on systems where dicom module
has not previously been run.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21120 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: provide option to wait for index completion

Since dicom indexer uses QFuture threading, it may return before
the database is ready.

Requires:

https://github.com/commontk/CTK/commit/868c4dbb552dbbc076a3ae7aa135e17be5dbfba2

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21121 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: provide access to new dicom indexer API

Addresses #2516 and #2618

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21122 3bd1e089-480b-0410-8dfb-8563597acbee
BUG 2546. Don't save storable nodes in the Main Scene View since they either in the main scene or are deleted on purpose.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21123 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Fix leak in liver and lung tests

don't use GetNodesByClass (vtkCollection) api to access scene

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21124 3bd1e089-480b-0410-8dfb-8563597acbee