View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002108 | Slicer4 | Core: GUI | public | 2012-05-23 10:54 | 2012-10-06 05:31 |
Reporter | lassoan | Assigned To | nicole | ||
Priority | normal | Severity | crash | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.1.0 | ||||
Target Version | Slicer 4.2.0 | Fixed in Version | Slicer 4.1.0 | ||
Summary | 0002108: 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). | ||||
Tags | No tags attached. | ||||
related to | 0002428 | closed | finetjul | Model maker creates less models than the number of labels in the input image |
related to | 0002429 | closed | nicole | An invalid hierarchy node in the Models module after deleting a model node in the Data module |
related to | 0002430 | closed | finetjul | Models jump around in the MRML tree view when visibility is changed |
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? |
|
Added the ability to delete model hierarchies via svn 20821-20823. No crashes observed. |
|
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 model maker again, click Apply =>All the models are correctly appear Delete models from the list (I couldn't reproduce this, just happened once) *Exit slicer Restart slicer, recreate the models *Click on the checkbox at the "Modelhierarchy_5" node |
|
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. |
|
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. |
|
Thanks, I'm going to resolve this one now and will update the others as I work on those issues. |
|
Sorry, it's crashing again. How to reproduce:
Tested on Slicer-4.1.0-2012-09-03-win-amd64.exe. |
|
Are you creating the models via the editor + model maker? |
|
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. |
|
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. |
|
svn 20923 adds an update to the GUI tree after batch processing the deletion of multiple models ends. |
|
Thank you it works correctly now. Tested on latest nightly Slicer-4.1.0-2012-10-05-win-amd64. |
|
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 |