MantisBT - ParaView
View Issue Details
0007238ParaView(No Category)public2008-06-25 05:242009-05-13 13:45
Stephane P 
Utkarsh Ayachit 
normalminoralways
closedfixed 
 
3.6 
ParaViS
0007238: bug with IceT with multiple views : the interval between views is not cleared
When using IceT with multiple views, there is a small gap between the views that is not cleared.
To reproduce it :
launch the server with pvserver -tdx=1, connect to it,
load the interval.pvsm state, and move the separation between the two views.
No tags attached.
? interval.pvsm (130,409) 2008-06-25 05:24
https://www.vtk.org/Bug/file/6562/interval.pvsm
diff BugFix-7238-Oct-9-2008.diff (24,361) 2008-10-10 15:55
https://www.vtk.org/Bug/file/6766/BugFix-7238-Oct-9-2008.diff
Issue History
2008-06-25 05:24Stephane PNew Issue
2008-06-25 05:24Stephane PFile Added: interval.pvsm
2008-06-25 09:56Ken MorelandNote Added: 0012521
2008-06-25 09:56Ken MorelandCategory3.4 => 3.6
2008-10-09 09:01Utkarsh AyachitStatusbacklog => tabled
2008-10-09 09:01Utkarsh AyachitAssigned To => Utkarsh Ayachit
2008-10-09 09:01Utkarsh AyachitAssigned ToUtkarsh Ayachit =>
2008-10-10 15:55Pat MarionFile Added: BugFix-7238-Oct-9-2008.diff
2008-10-10 15:56Pat MarionNote Added: 0013848
2009-02-17 18:56Ken MorelandCategory3.6 => Pending
2009-02-20 11:07Ken MorelandAssigned To => Utkarsh Ayachit
2009-02-20 11:07Ken MorelandCategoryPending => 3.6
2009-02-20 12:10Pat MarionNote Added: 0015256
2009-04-03 09:58Utkarsh AyachitNote Added: 0015897
2009-04-03 09:58Utkarsh AyachitStatustabled => @80@
2009-04-03 09:58Utkarsh AyachitResolutionopen => fixed
2009-04-15 16:56Alan ScottNote Added: 0016035
2009-04-15 16:56Alan ScottStatus@80@ => closed
2009-05-13 13:45Utkarsh AyachitTarget Version => 3.6
2011-06-16 13:10Zack GalbreathCategory => (No Category)
2011-06-23 11:44Utkarsh AyachitProject => ParaViS

Notes
(0012521)
Ken Moreland   
2008-06-25 09:56   
The gap between the views is an all around nuisance, especially when you have lots of views on a display much larger than the GUI. The gaps between views are proportional to those in the GUI. Because everything is scaled up the same, the gaps are often too large, especially in the vertical direction.

Rather than clear out the spaces, it would be nice to just make the views larger to abut each other. This would probably require the GUI to report view dimensions that extend into the GUI around its borders. Of course, that should only be done in tile display mode, not desktop delivery mode.
(0013848)
Pat Marion   
2008-10-10 15:56   
I've uploaded a bugfix patch, might not be ready for cvs until more testing is done.
(0015256)
Pat Marion   
2009-02-20 12:10   
In the bugfix patch, the main algorithm is pqMultiView::computeCompactSizes. This algorithm does what Ken describes above.

Everything else is just code to move the new size values to the server.
(0015897)
Utkarsh Ayachit   
2009-04-03 09:58   
Committing Pat's patch for collapsing gaps between views on the server side on a tile display.


/cvsroot/ParaView3/ParaView3/Qt/Components/pqMultiView.cxx,v <-- Qt/Components/pqMultiView.cxx
new revision: 1.20; previous revision: 1.19
/cvsroot/ParaView3/ParaView3/Qt/Components/pqMultiView.h,v <-- Qt/Components/pqMultiView.h
new revision: 1.14; previous revision: 1.13
/cvsroot/ParaView3/ParaView3/Qt/Components/pqViewManager.cxx,v <-- Qt/Components/pqViewManager.cxx
new revision: 1.68; previous revision: 1.67
/cvsroot/ParaView3/ParaView3/Qt/Components/pqViewManager.h,v <-- Qt/Components/pqViewManager.h
new revision: 1.26; previous revision: 1.25
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVDesktopDeliveryClient.cxx,v <-- Servers/Filters/vtkPVDesktopDeliveryClient.cxx
new revision: 1.10; previous revision: 1.9
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVDesktopDeliveryClient.h,v <-- Servers/Filters/vtkPVDesktopDeliveryClient.h
new revision: 1.7; previous revision: 1.6
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVDesktopDeliveryServer.cxx,v <-- Servers/Filters/vtkPVDesktopDeliveryServer.cxx
new revision: 1.14; previous revision: 1.13
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVDesktopDeliveryServer.h,v <-- Servers/Filters/vtkPVDesktopDeliveryServer.h
new revision: 1.11; previous revision: 1.10
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/Resources/rendering.xml,v <-- Servers/ServerManager/Resources/rendering.xml
new revision: 1.340; previous revision: 1.339
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMIceTMultiDisplayRenderViewProxy.cxx,v <-- Servers/ServerManager/vtkSMIceTMultiDisplayRenderViewProxy.cxx
new revision: 1.11; previous revision: 1.10
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMIceTMultiDisplayRenderViewProxy.h,v <-- Servers/ServerManager/vtkSMIceTMultiDisplayRenderViewProxy.h
new revision: 1.5; previous revision: 1.4
(0016035)
Alan Scott   
2009-04-15 16:56   
I think I understand the fix. If not, please reopen this bug.
Tested client/ remote server with the tdx switch listed below.