MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0004819ParaView(No Category)public2007-04-09 14:232009-05-13 13:47
ReporterBerk Geveci 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.6Fixed in Version 
Project
Topic Name
Type
Summary0004819: Cleanup progress handling
DescriptionProgress handling could use a rework. It should be done all in vtkPVProgressHandler. It should keep track of individual algorithm report intervals with timers and should report when the interval exceeds a configurable threshold.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0003925closed Berk Geveci Supporting abort & better progress handling 
related to 0005463closed Utkarsh Ayachit MPI features of vtkPVProgressHandler not used 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-04-09 12:00Berk GeveciNote Added: 0011206
2008-04-09 12:00Berk GeveciCategory => 3.6
2008-11-12 09:28Utkarsh AyachitRelationship addedrelated to 0003925
2008-11-12 09:28Utkarsh AyachitRelationship addedrelated to 0005463
2008-11-12 09:34Utkarsh AyachitNote Added: 0014091
2008-11-12 09:34Utkarsh AyachitStatustabled => @80@
2008-11-12 09:34Utkarsh AyachitResolutionopen => fixed
2008-11-19 18:24Alan ScottNote Added: 0014144
2008-11-19 18:24Alan ScottStatus@80@ => closed
2009-05-13 13:47Utkarsh AyachitTarget Version => 3.6
2011-06-16 13:10Zack GalbreathCategory => (No Category)

Notes
(0011206)
Berk Geveci   
2008-04-09 12:00   
We won't have time for this in 3.4.
(0014091)
Utkarsh Ayachit   
2008-11-12 09:34   
Cleaning progress handling. Rewrote vtkPVProgressHandler to ensure that it works in parallel. In the new design, CleanPendingProgress blocks till all processes are done reporting the progress thus ensuring that we don't miss out on progress events since the root is running faster than the satellites.

Also added a new vtkPVMPICommunicator which overrides Receive() to consume progress events as and when they appear. Hence if the root is blocking on some arbitrary receive eg. GatherInformation(), it will still get the progress events sent by the satellites and report them to the client.

/cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMViewProxy.cxx,v <-- Servers/ServerManager/vtkSMViewProxy.cxx
new revision: 1.20; previous revision: 1.19
/cvsroot/ParaView3/ParaView3/Servers/Common/CMakeLists.txt,v <-- Servers/Common/CMakeLists.txt
new revision: 1.84; previous revision: 1.83
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkClientConnection.cxx,v <-- Servers/Common/vtkClientConnection.cxx
new revision: 1.10; previous revision: 1.9
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkMPISelfConnection.cxx,v <-- Servers/Common/vtkMPISelfConnection.cxx
new revision: 1.7; previous revision: 1.6
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVMPICommunicator.cxx,v <-- Servers/Common/vtkPVMPICommunicator.cxx
initial revision: 1.1
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVMPICommunicator.h,v <-- Servers/Common/vtkPVMPICommunicator.h
initial revision: 1.1
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVProgressHandler.cxx,v <-- Servers/Common/vtkPVProgressHandler.cxx
new revision: 1.12; previous revision: 1.11
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVProgressHandler.h,v <-- Servers/Common/vtkPVProgressHandler.h
new revision: 1.3; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkProcessModule.cxx,v <-- Servers/Common/vtkProcessModule.cxx
new revision: 1.90; previous revision: 1.89
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkProcessModule.h,v <-- Servers/Common/vtkProcessModule.h
new revision: 1.68; previous revision: 1.67
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkProcessModuleConnection.cxx,v <-- Servers/Common/vtkProcessModuleConnection.cxx
new revision: 1.13; previous revision: 1.12
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkProcessModuleConnection.h,v <-- Servers/Common/vtkProcessModuleConnection.h
new revision: 1.10; previous revision: 1.9
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkSelfConnection.cxx,v <-- Servers/Common/vtkSelfConnection.cxx
new revision: 1.5; previous revision: 1.4
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkServerConnection.cxx,v <-- Servers/Common/vtkServerConnection.cxx
new revision: 1.16; previous revision: 1.15
(0014144)
Alan Scott   
2008-11-19 18:24   
YES!

This is orders of magnitude better than we had. Good job.
Tested client/server, 6 servers, dave's medium cth data, and with can.