View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0007238ParaView(No Category)public2008-06-25 05:242009-05-13 13:45
ReporterStephane P 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.6Fixed in Version 
Summary0007238: bug with IceT with multiple views : the interval between views is not cleared
DescriptionWhen 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.
TagsNo tags attached.
ProjectParaViS
Topic Name
Type
Attached Files? file icon interval.pvsm [^] (130,409 bytes) 2008-06-25 05:24
diff file icon BugFix-7238-Oct-9-2008.diff [^] (24,361 bytes) 2008-10-10 15:55 [Show Content]

 Relationships

  Notes
(0012521)
Ken Moreland (manager)
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 (reporter)
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 (reporter)
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 (administrator)
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 (manager)
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.

 Issue History
Date Modified Username Field Change
2008-06-25 05:24 Stephane P New Issue
2008-06-25 05:24 Stephane P File Added: interval.pvsm
2008-06-25 09:56 Ken Moreland Note Added: 0012521
2008-06-25 09:56 Ken Moreland Category 3.4 => 3.6
2008-10-09 09:01 Utkarsh Ayachit Status backlog => tabled
2008-10-09 09:01 Utkarsh Ayachit Assigned To => Utkarsh Ayachit
2008-10-09 09:01 Utkarsh Ayachit Assigned To Utkarsh Ayachit =>
2008-10-10 15:55 Pat Marion File Added: BugFix-7238-Oct-9-2008.diff
2008-10-10 15:56 Pat Marion Note Added: 0013848
2009-02-17 18:56 Ken Moreland Category 3.6 => Pending
2009-02-20 11:07 Ken Moreland Assigned To => Utkarsh Ayachit
2009-02-20 11:07 Ken Moreland Category Pending => 3.6
2009-02-20 12:10 Pat Marion Note Added: 0015256
2009-04-03 09:58 Utkarsh Ayachit Note Added: 0015897
2009-04-03 09:58 Utkarsh Ayachit Status tabled => @80@
2009-04-03 09:58 Utkarsh Ayachit Resolution open => fixed
2009-04-15 16:56 Alan Scott Note Added: 0016035
2009-04-15 16:56 Alan Scott Status @80@ => closed
2009-05-13 13:45 Utkarsh Ayachit Target Version => 3.6
2011-06-16 13:10 Zack Galbreath Category => (No Category)
2011-06-23 11:44 Utkarsh Ayachit Project => ParaViS


Copyright © 2000 - 2018 MantisBT Team