View Issue Details

IDProjectCategoryView StatusLast Update
0002849Slicer4Module Annotationspublic2020-01-22 12:14
Reporterjohan.andruejol Assigned Topinter  
Status assignedResolutionfixed 
Product VersionSlicer 4.2.2-1 
Target VersionSlicer 4.11.0Fixed in Version 
Summary0002849: Hierachy not respected when loading the same scene multiples times

When loading twice the same scene the nodes hierarchy is not of the loaded scene is not preserved the second time.
Save a scene with just a fiducial.
Import this scene.
Note that the new fiducial is added under the first "Fiducial list" with the original fiducial instead of the newly imported list.

Additional Information

It looks like the ParentNodeID in vtkMRMLHierarchyNode is not added as a referenced node ID in the scene. Thus the code in UpdateReferenceID regarding the ParentNodeID is never used.

I submitted a patch along.

TagsNo tags attached.


related to 0003462 closedjcfr ModelMaker functionality breaks display of Slice views 
child of 0004081 assignedjcfr Ensure reference are updated before NodeAddedEvent is invoked 


2013-01-03 10:52




2013-01-03 14:47

administrator   ~0007608

Last edited: 2013-01-04 03:44

Partially fixed in r21561:

The import of annotations is still broken, as part of r21561, a failing test has been created (vtkSlicerAnnotationModuleLogicImportSceneTest).



2013-01-04 03:47

administrator   ~0007609

Maybe Johan's solution is correct, vtkSlicerAnnotationModuleLogic::OnMRMLSceneNodeAdded() shouldn't do anything when the scene is in import/restore state.



2013-01-04 07:53

administrator   ~0007610

svn 21562[1] checks for scene restore/import before adding hierarchies.
That fixes importing a scene with fiducials, but rulers are still getting re-assigned to the newly imported ruler hierarchy.




2013-01-04 10:19

administrator   ~0007611

I was looking at the parent/associated node/child node ids and everything looked fine after importing a ruler scene, then I tried:

Open Slicer
Add a ruler
Save scene
Import scene

Look at the annotations module, the original ruler moved under the imported ruler list.
Double click on the second item in the second ruler list and rename it.
It jumps back to the first list.

Reimporting the file, only the first one jumps into the new ruler list, and it jumps back when I rename it. I suspect something in the scene model rather than the mrml code, so I'm reassigning this issue back to Julien.



2015-11-11 11:08

administrator   ~0013589

Fixed in r24723



2015-11-12 08:58

administrator   ~0013599

Reverted in r24734

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 959e9b16

2013-01-03 19:46:32


Details Diff
BUG: Fix import of hierarchy node with existing ID

Note that it does not fix the import of annotation hierarchies.
For some reasons (see vtkSlicerAnnotationModuleLogic::OnSceneNodeAdded),
the hierarchies are automatically loaded into the wrong hierarchy node
Issue 0002849

git-svn-id: 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Core/Testing/vtkMRMLSceneImportIDModelHierarchyConflictTest.cxx Diff File
mod - Libs/MRML/Core/vtkMRMLHierarchyNode.cxx Diff File
mod - Modules/Loadable/Annotations/Testing/Cxx/CMakeLists.txt Diff File
add - Modules/Loadable/Annotations/Testing/Cxx/vtkSlicerAnnotationModuleLogicImportSceneTest.cxx Diff File

Slicer: 2145-support-for-installing-extension-from-file 98ac3bf5

2013-01-04 12:51:54


Details Diff
BUG: don't add hierarchy nodes if the scene is importing/restoring. Issue 0002849

git-svn-id: 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/Loadable/Annotations/Logic/vtkSlicerAnnotationModuleLogic.cxx Diff File

Import 2017-06-07 23:51:09: master dd5c50f1

2015-11-11 15:04:19


Details Diff
BUG: Workaround MRML Scene import limitation. Fixes 0003462, 0002849 and 0004080

As documented in 0004081 [1], references to other nodes are not valid
when the NodeAddedEvent is invoked during the import of a scene in
an existing scene.

This prevented the qMRMLSceneModel from working well when it is
observing the scene and updating itself as new node are added
during import.

This commit workarounds the root issue by forcing all SceneModel
to have "lazyUpdate" enabled.

When lazyUpdate is enabled, the model ignores added node events when the
scene is importing/restoring, but synchronize with the scene once its

This affected a broad set of modules and was captured in the following

* 3462: ModelMaker functionality breaks display of Slice views
* 2849: Hierachy not respected when loading the same scene multiples times
* 4080: Transform module is not updated properly after importing a scene

Reviewed-by: Andras Lasso <>
Reviewed-by: Steve Pieper <>
Tested-by: Andras Lasso <>
Tested-by: Jean-Christophe Fillion-Robin <>

git-svn-id: 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Widgets/qMRMLSceneModel.cxx Diff File

Import 2017-06-07 23:51:09: master 7c91689f

2015-11-12 12:50:11


Details Diff
BUG: Revert workaround for MRML Scene import limitation. See 0004081

This commit reverts r24723 (BUG: Workaround MRML Scene import
limitation. Fixes 0003462, 0002849 and 0004080).

It turns out that this commit had unintended side effects. The following
tests were failing:


After concerting with the core team, since the correct fix involves
changes in the core of MRML, we decided to revert this commit and mark
the tickets 0003462, 0002849, 0004080 as known issues in the release note.

git-svn-id: 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Widgets/qMRMLSceneModel.cxx Diff File

Issue History

Date Modified Username Field Change
2013-01-03 10:52 johan.andruejol New Issue
2013-01-03 10:52 johan.andruejol Status new => assigned
2013-01-03 10:52 johan.andruejol Assigned To => pieper
2013-01-03 10:52 johan.andruejol File Added: FixHierarchyWhenImportingSceneSeveralTimes.patch
2013-01-03 14:43 finetjul Assigned To pieper => finetjul
2013-01-03 14:47 finetjul Note Added: 0007608
2013-01-03 14:47 finetjul Assigned To finetjul => nicole
2013-01-03 14:47 finetjul Category Base Code => Module Annotations
2013-01-04 03:44 finetjul Note Edited: 0007608
2013-01-04 03:47 finetjul Note Added: 0007609
2013-01-04 07:53 nicole Note Added: 0007610
2013-01-04 10:19 nicole Assigned To nicole => finetjul
2013-01-04 10:19 nicole Note Added: 0007611
2015-11-03 20:14 jcfr Relationship added related to 0003462
2015-11-03 20:14 jcfr Assigned To finetjul => jcfr
2015-11-03 20:14 jcfr Target Version => Slicer 4.5.0-1
2015-11-11 08:18 jcfr Relationship added child of 0004081
2015-11-11 11:08 jcfr Note Added: 0013589
2015-11-11 11:08 jcfr Status assigned => resolved
2015-11-11 11:08 jcfr Fixed in Version => Slicer 4.5.0-1
2015-11-11 11:08 jcfr Resolution open => fixed
2015-11-12 08:17 jcfr Status resolved => assigned
2015-11-12 08:17 jcfr Fixed in Version Slicer 4.5.0-1 =>
2015-11-12 08:17 jcfr Target Version Slicer 4.5.0-1 => Slicer 4.5.1
2015-11-12 08:58 jcfr Note Added: 0013599
2016-10-12 02:56 jcfr Target Version Slicer 4.5.1 => Slicer 4.7.0
2017-06-07 23:27 Changeset attached => Slicer 2145-support-for-installing-extension-from-file 98ac3bf5
2017-06-07 23:27 finetjul Changeset attached => Slicer 2145-support-for-installing-extension-from-file 959e9b16
2017-06-10 08:51 jcfr Changeset attached => Slicer master 7c91689f
2017-06-10 08:51 jcfr Changeset attached => Slicer master dd5c50f1
2017-09-27 11:13 lassoan Target Version Slicer 4.7.0 => Slicer 4.9.0
2018-03-22 01:33 jcfr Target Version Slicer 4.9.0 => Slicer 4.11.0
2020-01-22 12:14 jcfr Assigned To jcfr => pinter