MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0003290ParaView(No Category)public2006-05-25 14:592009-12-09 14:51
ReporterKen Moreland 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Topic Name
Type
Summary0003290: SNL: PVR: Ordered composite distributor runs too much during animation
DescriptionWhen creating an animation by clicking the "record next frame" button, the vtkOrderedCompositeDistributor unnecessarily runs on all of the parts. Also, while running an animation with only the camera change, the vtkOrderedCompositeDistributor is run on the parts.

The vtkOrderedCompositeDistributor, which can potentially take a long time, should only run if some geometry changes.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-12-09 14:51Berk GeveciProject@3@ => ParaView
2011-06-16 13:09Zack GalbreathCategory => (No Category)

Notes
(0004406)
Ken Moreland   
2006-07-05 13:27   
I found a workaround. If you turn off the geometry cache, that seems to solve the problem. However, that step should not be necessary.
(0005693)
Utkarsh Ayachit   
2006-11-10 14:28   
(1) Could not reproduce the re-excuting of distributor on clicking "record next frame".

(2)
The distributor to is connected after the UpdateSuppressor (which keeps the cache).

When animating without cache, the UpdateSuppressor "updates" only when some property on the connected source changes in which case the distributor is corretly re-executed. This does not happen when the animation merely contains camera changes etc.

When animating with cache, the UpdateSuppressor genrates a new output data for every step in the animation. In reality the data may not have changed at all, however the update suppressor cannot know (nor can the display proxy). Hence, in vtkSMCompositeDisplayProxy::CacheUpdate() the distributed geometry is marked invalid, as a result the ordered compositor re-executes.
(0005709)
Utkarsh Ayachit   
2006-11-13 17:06   
(2) can be fixed by using the UpdateSuppressor after the Distributor for caching instead of the one before it. That when when no geometry changes, the distributor won't be executed.
(0005710)
   
2006-11-14 08:29   
Cache is kept by update suppressors after the distributor (if any) that way when no geometry changes when using cache, the distributor is not executed." Servers/ServerManager
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMCompositeDisplayProxy.cxx,v <-- vtkSMCompositeDisplayProxy.cxx
new revision: 1.26; previous revision: 1.25
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMDataObjectDisplayProxy.cxx,v <-- vtkSMDataObjectDisplayProxy.cxx
new revision: 1.24; previous revision: 1.23
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMDataObjectDisplayProxy.h,v <-- vtkSMDataObjectDisplayProxy.h
new revision: 1.18; previous revision: 1.17
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMLODDisplayProxy.cxx,v <-- vtkSMLODDisplayProxy.cxx
new revision: 1.14; previous revision: 1.13
(0005711)
Utkarsh Ayachit   
2006-11-14 08:29   
Cache is kept by update suppressors after the distributor (if any) that way when no geometry changes when using cache, the distributor is not executed." Servers/ServerManager
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMCompositeDisplayProxy.cxx,v <-- vtkSMCompositeDisplayProxy.cxx
new revision: 1.26; previous revision: 1.25
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMDataObjectDisplayProxy.cxx,v <-- vtkSMDataObjectDisplayProxy.cxx
new revision: 1.24; previous revision: 1.23
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMDataObjectDisplayProxy.h,v <-- vtkSMDataObjectDisplayProxy.h
new revision: 1.18; previous revision: 1.17
/cvsroot/ParaView/ParaView/Servers/ServerManager/vtkSMLODDisplayProxy.cxx,v <-- vtkSMLODDisplayProxy.cxx
new revision: 1.14; previous revision: 1.13
(0005742)
Ken Moreland   
2006-11-20 13:50   
The ordered composite distributor seems to run much less frequently now. Are we ready to close this bug out?