BUG: Fix crash introduced by commit r21160

The problem is that Interactor member is not initialized for Displayable
manager that don't have at list on Interactor event to observe.

Before the change, the interactor was returned using
GetRenderWindow()->GetInteractor()

After the change, this->Interactor is returned.

This commit ensures that Interactor variable is set when the Render
is also set.

See https://github.com/Slicer/Slicer/commit/1264dac8ca9aaeba54627b8f8a37333c0f5f8a49#L0R648

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21165 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: include entire scalar range in threshold

Because the ctk range widget rounds the values for display, it
can have the effect of truncating the values such that the
selected range does not include the highest
value(s) in the volume.  Adding 1 to the max value avoids this
problem with no anticipated downsides.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21166 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: 2321 Should not check on Mac machines

The linker ld for OSX does not have --as-needed and --no-as-needed
flags.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21167 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Ensure extension description file are generated in top level build dir

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21168 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Export qSlicerBaseQTCLIPythonD library so that extensions can link it

See http://slicer-devel.65872.n3.nabble.com/Build-error-in-SlicerRT-extension-cannot-open-input-file-qSlicerBaseQTCLIPythonD-lib-tc4026390.html

Thanks to Csaba for reporting the issue.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21169 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: 2632. GrowCut crashes with input volume of type char.

The intensity image is assigned to the first input on the
vtkITKGrowCutImageFilter which forces the output segmentation
image to have the same pixel type.  But the output segmetentation
is assumed at the ITK level to be same pixel type as the gesture
image.

Overrode ExecuteInformation() so information from the second
input is propagated to the output image, so the gesture image
and output segmentation image have the same pixel type.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21170 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: avoid over-responding to mrml scene changes in per-structure volumes

Use the specific methods for node added and remove events rather
than the generic modified events. This avoids creating/destroying
Qt classes while the scene is being torn down and avoids a
crash in the ThresholdThreading test.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21171 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: avoid updating Qt while scene is batch processing

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21172 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: provide feedback during the editor test

Make sure qt has had time to process events before teardown of the
userinterface - otherwise pythonqt may hold references to QStandardItem
instances (and perhaps other objects) that Qt has destroyed.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21173 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fix for issue #2514

Make sure the widgets exist before updating the layout.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21174 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: reduced number of Modified events in Interactive Tractography module

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21175 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fix scene view crashes in 2514

The layout node in the layout logic was not cleared when a scene
was restored, which meant that it was out of date with respect
to the current scene and led to a crash.  Now we explicitly unreference
it when a scene restore begins and then re-access it after
the scene has been restored.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21176 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: avoid calling non-test method during unittest

The python unittest framework considers any test* method to be
a test and calls it with no arguments.  By renaming the helper
method to be perform_ rather than test_ we let the test
succeed.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21177 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: re-enable all atlas tests for the general testing case

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21178 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Add qMRMLAnnotationTreeView into Qt designer plugin

It can then be used when designing your module with Qt Designer

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21179 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix compilation warnings in vtkMRML

Unused variable for vtkMRMLModelSliceDisplayableManagerTest
Automatic cast for vtkMRMLFreeSurferModelOverlayStorageNode

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21180 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Expose Annotations displayable managers to other modules

By moving the displayable managers into their own library,
it is then possible for modules to use and link to it.
The Annotations VTKWidgets had to be moved as well.

Issue #2606

Conflicts:
	Modules/Loadable/Annotations/CMakeLists.txt

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21181 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: update the code to use the new ROI visibility API


git-svn-id: http://svn.slicer.org/Slicer4/trunk@21182 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Annotations displayable manager depends on annotation logic

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21183 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fix for #2605 - special characters in data bundle filenames

Since data bundles must sometimes make filenames on-the-fly
based on the node names, add a step to ensure filenames
do not contain special characters (like colon or slash).

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