View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003955||Slicer4||Core: Performance||public||2015-02-09 12:37||2015-02-17 10:57|
|Product Version||Slicer 4.4.0|
|Target Version||Fixed in Version|
|Summary||0003955: Store color nodes in a private MRML scene|
One of the main performance issues in Slicer is that in case of high number of nodes and visited modules, all the scene models that have been instantiated iterate over all the MRML nodes in the scene.
The problem is that right after startup there are 109 nodes, 66 of which are color nodes. This issue could be solved relatively easily by handling the color nodes in the same way the volume rendering presets are handled: in a private MRML scene, from which the nodes are transferred in the Slicer MRML scene only on demand.
|Tags||No tags attached.|
Reminder sent to: nicole
Assuming we apply this approach, do you anticipate any issue with existing scene (SPL Atlas, ...) ?
The idea would be teach the xml parser to ignore nodes that match the properties of the built-in one ..
I think the issue to be careful of will be with user loaded color nodes. They need to be saved with the scene and restored properly.
Actually, there are 80 color nodes and 25 other nodes. So, storing the unused color nodes in the scene increases the number of nodes by more than a factor of 3!
Note from Slicer Dev Hangout Feb 18: consider making a new subclass color node mrml combo box that has a second private scene and can copy nodes over the main scene?
EDIT (Jc): See also http://www.slicer.org/slicerWiki/index.php/Developer_Meetings/20150217
|2015-02-09 12:37||pinter||New Issue|
|2015-02-09 12:37||pinter||Status||new => assigned|
|2015-02-09 12:37||pinter||Assigned To||=> jcfr|
|2015-02-09 12:40||jcfr||Status||assigned => acknowledged|
|2015-02-09 12:47||jcfr||Note Added: 0012902|
|2015-02-09 13:35||nicole||Note Added: 0012903|
|2015-02-16 12:04||lassoan||Note Added: 0012915|
|2015-02-17 10:57||nicole||Note Added: 0012917|
|2015-02-17 11:20||jcfr||Note Edited: 0012917|