View Issue Details Jump to Notes ] Print ]
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 
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.
TagsNo tags attached.
Project
Topic Name
Type
Attached Files

 Relationships

  Notes
(0004406)
Ken Moreland (manager)
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 (administrator)
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 (administrator)
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 (administrator)
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 (manager)
2006-11-20 13:50

The ordered composite distributor seems to run much less frequently now. Are we ready to close this bug out?

 Issue History
Date Modified Username Field Change
2009-12-09 14:51 Berk Geveci Project @3@ => ParaView
2011-06-16 13:09 Zack Galbreath Category => (No Category)


Copyright © 2000 - 2018 MantisBT Team