View Issue Details

IDProjectCategoryView StatusLast Update
0002108Slicer4Core: GUIpublic2012-10-06 05:31
Reporterlassoan Assigned Tonicole  
PrioritynormalSeveritycrashReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.1.0 
Target VersionSlicer 4.2.0Fixed in VersionSlicer 4.1.0 
Summary0002108: Crash when deleting a model after deleting its parent hierarchy node
Description

When a model hierarchy node is deleted then the models below the node are not deleted from the scene => I would consider this an error, as usually whatever operation you apply on the parent node is applied on the child nodes. Also, it is easy to create a set of models are created in a single step, but currently there is no way to delete the models in one step.

Slicer crashes after trying to delete any node that was part of the deleted hierarchy.

It is reproducible with the latest nightly build (Slicer-4.1.0-2012-05-20-win-amd64.exe).

TagsNo tags attached.

Relationships

related to 0002428 closedfinetjul Model maker creates less models than the number of labels in the input image 
related to 0002429 closednicole An invalid hierarchy node in the Models module after deleting a model node in the Data module 
related to 0002430 closedfinetjul Models jump around in the MRML tree view when visibility is changed 

Activities

nicole

nicole

2012-08-20 07:58

administrator   ~0005574

June 13/12 I added the delete multiple models option, as well as checking that hierarchies get deleted properly. Can you double check that this functionality is what you were requesting?

nicole

nicole

2012-08-22 08:17

administrator   ~0005699

Added the ability to delete model hierarchies via svn 20821-20823. No crashes observed.

lassoan

lassoan

2012-08-22 11:17

developer   ~0005714

There are no crashes any more, but there are a couple of new issues (latest trunk version, on Win7, 64-bit, release mode).

How to reproduce:

Load MRHead sample
Go to the editor module, draw 6 structures with different colors
*Go to the model maker, create a new hierarchy, generate the models
=> ERROR: Only 3 models are visible (the 1st, 2nd, and 3rd), the 4th, 5th, 6th models are generated with the correct colors, but with incorrect geometry (the same as the 1st, 2nd, and 3rd)

Go to model maker again, click Apply =>All the models are correctly appear
Go to Models
Turn off the visibility of the first listed model under Models (Model_4_connective_tissue)
=> ERROR: it jumps to the second place (below Model_5_blood)
Turn off the visibility of the first listed model (Model_5_blood)
=> ERROR: it jumps the third place in the list

Delete models from the list (I couldn't reproduce this, just happened once)
Hide all models
=>ERROR: the deleted model is still visible in the 3D viewer (in gray)

*Exit slicer
=>ERROR: there are leaks

Restart slicer, recreate the models
Go to the Data module, delete a model
*Go to models module
=>ERROR: instead of the seeing only 5 models, there are 5 models and a "Modelhierarchy_5" node listed under the Models hierarchy node (it seems that the model deleted in the data module has been converted to a model hierarchy node)

*Click on the checkbox at the "Modelhierarchy_5" node
=>ERROR: the "Model_2_bone" model jumps into the "Modelhierarchy_5" hierarchy

nicole

nicole

2012-08-22 11:31

administrator   ~0005719

The delete in the data module doesn't delete hierarchy nodes (it just deletes the selected node, not any heirarchies it may be associated with), that's why I put the hierarchy deletion in the models module.

The jumping around of the entries in the list isn't related to deleting, but to the scene models and tree views.

Could you please put the new bugs into new bug reports and make them related to this one (the model maker ones esp.)? It's easier to track and fix them that way.

lassoan

lassoan

2012-08-22 13:07

developer   ~0005728

Added 3 related issues (0002428, 0002429, 0002430). The only issues that I haven't included in these are the memory leak on quit and the hard-to-reproduce remaining model in the 3D viewer.

nicole

nicole

2012-08-22 13:24

administrator   ~0005734

Thanks, I'm going to resolve this one now and will update the others as I work on those issues.

lassoan

lassoan

2012-09-04 08:57

developer   ~0005944

Sorry, it's crashing again.

How to reproduce:

  • Create 5 models
  • Go to Models module
  • Create a hierarchy, drag&drop two models
  • Right-click, choose delete the hierarchy node, click OK to delete all children
    => ERROR: the children and the hierarchy node still visible in the tree
  • Click on the hierarchy node or children that was supposed to be deleted
    => ERROR: Slicer crashes

Tested on Slicer-4.1.0-2012-09-03-win-amd64.exe.

nicole

nicole

2012-09-04 09:57

administrator   ~0005948

Are you creating the models via the editor + model maker?

lassoan

lassoan

2012-09-04 10:04

developer   ~0005949

It does not seem to matter how you create the models. It's reproducible by simply loading a couple of models from files or creating them one by one from the editor.

nicole

nicole

2012-09-04 11:10

administrator   ~0005953

Hmm, odd, the crash goes away if I turn off the batch processing settings around the multiple delete (there's a lazy upate flag that's turned on). Let me see if I can get it working with the batch processing flags set, that helps with speed ups, but if not, I'll check in that change since it'll avoid the crash.

nicole

nicole

2012-09-04 11:36

administrator   ~0005958

svn 20923 adds an update to the GUI tree after batch processing the deletion of multiple models ends.

lassoan

lassoan

2012-10-06 05:31

developer   ~0006418

Thank you it works correctly now. Tested on latest nightly Slicer-4.1.0-2012-10-05-win-amd64.

Issue History

Date Modified Username Field Change
2012-05-23 10:54 lassoan New Issue
2012-08-20 07:58 nicole Note Added: 0005574
2012-08-20 07:58 nicole Assigned To => nicole
2012-08-20 07:58 nicole Status new => assigned
2012-08-20 07:58 nicole Target Version => Slicer 4.2.0 - Feature freeze Sept 1st 2012
2012-08-22 08:17 nicole Note Added: 0005699
2012-08-22 08:17 nicole Status assigned => resolved
2012-08-22 08:17 nicole Fixed in Version => Slicer 4.1.0
2012-08-22 08:17 nicole Resolution open => fixed
2012-08-22 11:17 lassoan Note Added: 0005714
2012-08-22 11:17 lassoan Status resolved => feedback
2012-08-22 11:17 lassoan Resolution fixed => reopened
2012-08-22 11:31 nicole Note Added: 0005719
2012-08-22 11:46 lassoan Relationship added related to 0002428
2012-08-22 11:52 lassoan Relationship added related to 0002429
2012-08-22 13:06 lassoan Relationship added related to 0002430
2012-08-22 13:07 lassoan Note Added: 0005728
2012-08-22 13:24 nicole Note Added: 0005734
2012-08-22 13:24 nicole Status feedback => resolved
2012-08-22 13:24 nicole Resolution reopened => fixed
2012-09-04 08:57 lassoan Note Added: 0005944
2012-09-04 08:57 lassoan Status resolved => feedback
2012-09-04 08:57 lassoan Resolution fixed => reopened
2012-09-04 09:57 nicole Note Added: 0005948
2012-09-04 10:04 lassoan Note Added: 0005949
2012-09-04 11:10 nicole Note Added: 0005953
2012-09-04 11:36 nicole Note Added: 0005958
2012-09-04 11:36 nicole Status feedback => resolved
2012-09-04 11:36 nicole Resolution reopened => fixed
2012-10-06 05:31 lassoan Note Added: 0006418
2012-10-06 05:31 lassoan Status resolved => closed