View Issue Details

IDProjectCategoryView StatusLast Update
0001396Slicer4Core: GUIpublic2012-07-28 09:54
Reporterpieper Assigned Tofinetjul  
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.0.0 
Target VersionSlicer 4.0.0Fixed in VersionSlicer 4.0.0 
Summary0001396: slow scene loading when ctkVTKThumbnailWidget is enabled
Description

This scene loads very slowly in slicer4:

/home/pieper/Dropbox/RonsExamples/FGBG-Test/SecondScene.mrml

After some profiling it seems a lot of time is bring spent in: ctkVTKThumbnailViewPrivate::updateBounds()

Stubbing this out (see diff below) makes the scene load very quickly!

Can we turn off this widget during scene loads?

Additional Information

Here's the part to stub out:

pieper@flint:~/slicer4/latest/Slicer4-superbuild$ (cd CTK; git diff)
diff --git a/Libs/Visualization/VTK/Widgets/ctkVTKThumbnailView.cpp b/Libs/Visua
index ae83f65..9d98713 100644
--- a/Libs/Visualization/VTK/Widgets/ctkVTKThumbnailView.cpp
+++ b/Libs/Visualization/VTK/Widgets/ctkVTKThumbnailView.cpp
@@ -149,6 +149,7 @@ void ctkVTKThumbnailViewPrivate::initCamera()
//---------------------------------------------------------------------------
void ctkVTKThumbnailViewPrivate::updateBounds()
{

  • return;
    Q_Q(ctkVTKThumbnailView);
    vtkRenderer* ren = this->Renderer;
TagsNo tags attached.

Relationships

related to 0001407 closedfinetjul loading scenes is very slow 
related to 0001229 closedfinetjul loading scene with fg/and bg does not work 

Activities

finetjul

finetjul

2011-09-07 21:43

administrator   ~0002940

My first discovery is that a polydata needs to update its entire pipeline (not just the update_extent pass) to know it's bounds. This would explain why the navigation view requires glyph computation.
What I still don't understand is why the computation would be done more than once.

finetjul

finetjul

2011-09-12 12:36

administrator   ~0002984

Fixed in r18029:
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=18029

jcfr

jcfr

2012-07-27 15:36

administrator   ~0005313

Dear issue reporter,

Good news :) Slicer developers SOLVED the problem you reported - YOU now need to VERIFY and CLOSE this issue.

Issue History

Date Modified Username Field Change
2011-09-06 11:41 pieper New Issue
2011-09-06 11:41 pieper Status new => assigned
2011-09-06 11:41 pieper Assigned To => finetjul
2011-09-07 21:43 finetjul Note Added: 0002940
2011-09-10 23:10 finetjul Priority normal => high
2011-09-12 05:50 finetjul Relationship added related to 0001337
2011-09-12 12:30 finetjul Relationship deleted related to 0001337
2011-09-12 12:31 finetjul Relationship added related to 0001407
2011-09-12 12:36 finetjul Note Added: 0002984
2011-09-12 12:36 finetjul Status assigned => resolved
2011-09-12 12:36 finetjul Fixed in Version => Slicer 4.0 RSNA
2011-09-12 12:36 finetjul Resolution open => fixed
2011-09-14 20:30 finetjul Relationship added related to 0001229
2012-07-27 15:36 jcfr Note Added: 0005313
2012-07-28 09:54 pieper Status resolved => closed