View Issue Details

IDProjectCategoryView StatusLast Update
0002528Slicer4(No Category)public2013-05-31 09:29
Reportergabriella Assigned Toalexy  
PrioritynormalSeveritycrashReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.1.0 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.2.1 
Summary0002528: Slicer crashes when LUT changed
Description

In Volume Module under Display tab change Scalar Mode to maxEigenvalue then LUT to DiffusionTensorDisplayProperties; Slicer always crashes with following printout in shell:
"Diffusion Tensor Scalar Measurements completed without errors

Show module (name): "Volumes"
error: [/gabi/szatmary/SLICER/Slicer4nightly/./bin/SlicerApp-real] exit abnormally - Report the problem."

I use linux 64bit and Slicer4.1.0-2012-09-11 Nightly.

thanks
Gabriella

TagsNo tags attached.

Relationships

parent of 0002796 closednicole PET Lookup table is missing from 4.2.1 
related to 0003141 closedalexy crash when saving scene (null lookup table in color table storage node) 

Activities

alexy

alexy

2012-11-02 09:03

developer   ~0007019

Last edited: 2013-05-14 12:30

Excluded derived ColorTable nodes (such as vtkMRMLDiffusionTensorDisplayPropertiesNode) from the LUT selection since they may not contain LUT's

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21306

nicole

nicole

2013-03-14 12:54

administrator   ~0008123

The FreeSurfer procedural color nodes are missing from the Volumes module, Display, Lookup Table drop down menu.
The nodes appear in the Colors module and it would be nice to be able to use them for volumes.
Could we make the hiding of child node types finer grained through the use of the hideChildNodeTypes list so that the Freesurfer nodes can still be used while the non LUT bearing ones are excluded?

bluered = getNode("vtkMRMLFreeSurferProceduralColorNodeBlueRed")

lut = bluered.GetLookupTable()

print lut

shows a valid look up table.

From the dev mailing list:
PET lookup tables are not accessible any more in Volumes

jcfr

jcfr

2013-03-19 11:01

administrator   ~0008158

This will be discussed during MArch 19th 2013 Developer Hangout. See http://www.slicer.org/slicerWiki/index.php/Developer_Meetings/20130319

nicole

nicole

2013-05-14 11:47

administrator   ~0008639

In
Slicer/Modules/Loadable/Volumes/Widgets/Resources/UI/qSlicerScalarVolumeDisplayWidget.ui
the ColorTableComboBox has it's property showChildNodeTypes set to false.

nicole

nicole

2013-05-14 12:42

administrator   ~0008640

Last edited: 2013-05-14 12:45

Looking at the color node tree:
http://slicer.org/doc/html/classvtkMRMLColorNode.html
we don't want to allow selecting vtkMRMLDiffusionTensorDisplayPropertiesNode or vtkMRMLGlyphableVolumeDisplayPropertiesNode nodes but we do want to be able to select procedural color nodes.

jcfr

jcfr

2013-05-14 13:37

administrator   ~0008645

As discussed during Developer hangout of May 14th, here is the proposed action plan:

1) Revert commit 21306

  • Make the code robust to prevent the crash

    2) Update ColorTableComboBox to exclude unwanted child type

    3) Eventually, explore how the class hierarchy could be updated to avoid the workaround (2)

pieper

pieper

2013-05-14 15:18

administrator   ~0008646

As discussed in the developer hangout [1] we should do the following:

1 - revert the change so that the the pet and other color nodes show up in the volumes module

2 - figure out why these can cause a crash when trying to display the result of the diffusion tensor scalars and fix the crash

3 - look into adding attributes to the glyphable and diffusion tensor color nodes so they are hidden from the user in the Volumes module while other color table nodes and subclasses are displayed.

http://www.slicer.org/slicerWiki/index.php/Developer_Meetings/20130514#To_discuss

alexy

alexy

2013-05-16 11:03

developer   ~0008654

Last edited: 2013-05-16 11:04

Fixed the cause of the crash because of invalid LUT in vtkMRMLDiffusionTensorDisplayPropertiesNode

At revision: 22006 - http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=22006

nicole

nicole

2013-05-31 09:29

administrator   ~0008695

Thanks!

Issue History

Date Modified Username Field Change
2012-09-15 10:46 gabriella New Issue
2012-09-15 10:46 gabriella Status new => assigned
2012-09-15 10:46 gabriella Assigned To => demian
2012-10-30 10:38 pieper Assigned To demian => alexy
2012-11-02 09:03 alexy Note Added: 0007019
2012-11-02 09:03 alexy Status assigned => resolved
2012-11-02 09:03 alexy Resolution open => fixed
2012-12-08 09:49 jcfr Fixed in Version => Slicer 4.2.1
2012-12-08 09:49 jcfr Target Version => Slicer 4.2.0
2013-03-14 12:54 nicole Note Added: 0008123
2013-03-14 12:54 nicole Status resolved => feedback
2013-03-14 12:54 nicole Resolution fixed => reopened
2013-03-19 11:01 jcfr Note Added: 0008158
2013-04-29 10:41 nicole Relationship added parent of 0002796
2013-05-14 11:47 nicole Note Added: 0008639
2013-05-14 12:30 jcfr Note Edited: 0007019
2013-05-14 12:42 nicole Note Added: 0008640
2013-05-14 12:45 nicole Note Edited: 0008640
2013-05-14 13:37 jcfr Note Added: 0008645
2013-05-14 14:31 jcfr Status feedback => assigned
2013-05-14 14:31 jcfr Target Version Slicer 4.2.0 => Slicer 4.3.0
2013-05-14 15:18 pieper Note Added: 0008646
2013-05-14 15:18 pieper Status assigned => acknowledged
2013-05-16 11:03 alexy Note Added: 0008654
2013-05-16 11:03 alexy Status acknowledged => resolved
2013-05-16 11:03 alexy Resolution reopened => fixed
2013-05-16 11:04 jcfr Note Edited: 0008654
2013-05-31 09:29 nicole Note Added: 0008695
2013-05-31 09:29 nicole Status resolved => closed
2013-06-03 13:36 pieper Relationship added related to 0003141
2016-02-29 13:56 jcfr Category Module Diffusion => (No Category)