View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0013130 | ParaView | (No Category) | public | 2012-04-25 14:41 | 2015-09-03 05:41 | ||||
Reporter | Alan Scott | ||||||||
Assigned To | Cory Quammen | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | 3.14 | ||||||||
Target Version | 4.4 | Fixed in Version | 4.4 | ||||||
Summary | 0013130: color legend layout should allow manual control | ||||||||
Description | We need to add the ability to manually set the color legend to be horizontal or vertical. Currently this is automatic, which is problematic if you need numerous, horizontal color legends on the right side of the screen. | ||||||||
Tags | ColorMap | ||||||||
Project | Sandia | ||||||||
Topic Name | 13130_fix_scalar_bar_settings | ||||||||
Type | feature | ||||||||
Attached Files | |||||||||
Relationships | |||||||||||
|
Relationships |
Notes | |
(0034686) Alan Scott (manager) 2015-07-07 22:23 |
Note that the use case my user has given is as follows: Split view horizontal. Split each view vertically. Please the can in each of these 4 views. Paint by something (Vel). Now, try to change each color legend to be full height, right side of the views. Make them exactly the same size. Increase the annotation/label text size. |
(0034775) David C. Lonie (developer) 2015-07-14 13:20 |
Pull requests are in: VTK: https://gitlab.kitware.com/vtk/vtk/merge_requests/423 [^] PV: https://gitlab.kitware.com/paraview/paraview/merge_requests/221 [^] |
(0034825) Utkarsh Ayachit (administrator) 2015-07-27 11:27 |
The Edit Color Legent Paramemeters now has checkbox that can be used to decide whether auto-orientation is to be used. If unchecked, user can override how the color legend should be oriented. |
(0034828) Alan Scott (manager) 2015-07-27 18:11 |
I don't think this works well at all. Here is how I tested: * Linux, master, local server. * Delete your .config files, don't run -dr * Open can.exo. All vars on. Apply. * Color by ACCL. * Edit Color Map, which brings up the Edit Map Color. (As an asside, please make these the same. You don't Edit Color Map to bring up the Edit Map Color dialog!) * Edit Color Legend Properties. Unselect Auto Orient. Orientation Horizontal. Apply. * Save Current settings values as default. Now, say OK. <<Bug>> - the horizontal color legend is half cut off, off the lower right of the screen. For horizontal, please center it. * Close ParaView. Open ParaView. * Open can.exo. All vars on. Apply. * Color by ACCL. <<Bug>> - The color legend is vertical. We didn't save the current settings as default. * Edit Color Legend Properties. Unselect Auto Orient. Orientation Horizontal. Apply. * Save Current settings values as default. Now, say OK. * Split screen horizontal. Make the can visible. Color by ACCL. <<Bug>> - We get the following error: ERROR: In /projects/viz/paraview/src/ParaView/4.4.0-junk2/ParaView4/VTK/Rendering/Core/vtkCoordinate.cxx, line 351 vtkCoordinate (0x7c26c30): Request for coordinate transformation without required viewport ERROR: In /projects/viz/paraview/src/ParaView/4.4.0-junk2/ParaView4/VTK/Rendering/Core/vtkCoordinate.cxx, line 351 vtkCoordinate (0x7c26e30): Request for coordinate transformation without required viewport <<Bug>> - The color legend is vertical! |
(0034845) David C. Lonie (developer) 2015-07-28 09:59 |
Reassigning to Utkarsh, he's got some plans for these issues. BTW, this patch fixes the vtkCoordinate warnings: --- Interaction/Widgets/vtkBorderRepresentation.cxx +++ Interaction/Widgets/vtkBorderRepresentation.cxx @@ -471,9 +471,10 @@ void vtkBorderRepresentation::UpdateShowBorder() //------------------------------------------------------------------------- void vtkBorderRepresentation::BuildRepresentation() { - if ( this->GetMTime() > this->BuildTime || - (this->Renderer && this->Renderer->GetVTKWindow() && - this->Renderer->GetVTKWindow()->GetMTime() > this->BuildTime) ) + if (this->Renderer && + (this->GetMTime() > this->BuildTime || + (this->Renderer->GetVTKWindow() && + this->Renderer->GetVTKWindow()->GetMTime() > this->BuildTime))) { // Negotiate with subclasses if ( ! this->Negotiated ) |
(0034937) Utkarsh Ayachit (administrator) 2015-08-10 11:12 |
Cory, Mind taking care of the warning message? Thanks |
(0034938) David C. Lonie (developer) 2015-08-11 08:53 |
Warning PR is up: VTK: https://gitlab.kitware.com/vtk/vtk/merge_requests/519 [^] PV: https://gitlab.kitware.com/paraview/paraview/merge_requests/293 [^] |
(0034940) Utkarsh Ayachit (administrator) 2015-08-11 11:15 |
The warning has been fixed. BUG 0015629 has been reported for the stickness issue. |
(0034962) Alan Scott (manager) 2015-08-11 18:02 |
Nope, this still doesn't work correctly. See my long note, see the <<bug>> sections. Basically, it doesn't remember legend layout between sessions, even though it was told to. Main example is to remember horizontal layout of the color legend. |
(0034993) Cory Quammen (developer) 2015-08-13 16:58 |
Regarding the Orientation not being restored from settings: The Orientation property of the ScalarBarRepresentation has an information property. As such, when vtkSMParaViewPipelineController::PostInitializeProxy() goes to reset the value to the domain defaults, the value of the information property (which is the current value of the orientation) is set as the orientation value. This should be fixable by adding a <NoDefault /> hint to the orientation property. |
(0034998) Cory Quammen (developer) 2015-08-15 09:31 |
https://gitlab.kitware.com/paraview/paraview/merge_requests/301 [^] commit 97d854b224bc0fd4bfa29b5d172f16323496f59d Author: Cory Quammen <cory.quammen@kitware.com> Date: Sat Aug 15 09:19:33 2015 -0400 BUG: 13130 fixed scalar bar settings The AutoOrient and Orientation seetetings were not being restored from the settings file. Since these have an information property, they were being reset to the value of the information property. Added a <NoDefault /> hint to stop this from happening. Also, changed the position of scalar bars oriented horizontally when they are initially added to keep them in the view. |
(0035025) Cory Quammen (developer) 2015-08-18 23:49 |
This change has been merged into master. |
(0035027) Utkarsh Ayachit (administrator) 2015-08-19 08:09 |
The "warning" fix is in VTK, it will be brought into ParaView when we update the VTK reference (which should be soon). |
(0035137) Alan Scott (manager) 2015-09-02 16:53 |
Yep, this is fixed correctly. Note that when the color legend originally moved from vertical to horizontal, it was cut off. That's probably OK. After that, ParaView correctly placed it on the screen. Tested local server, Linux, master. Used my (long winded) testing notes above. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2012-04-25 14:41 | Alan Scott | New Issue | |
2013-07-26 12:35 | Utkarsh Ayachit | Tag Attached: ColorMap | |
2014-01-16 17:05 | Utkarsh Ayachit | Target Version | => 4.2 |
2014-11-14 22:52 | Utkarsh Ayachit | Target Version | 4.2 => 4.4 |
2015-06-08 14:29 | Utkarsh Ayachit | Assigned To | => David C. Lonie |
2015-06-08 14:29 | Utkarsh Ayachit | Status | backlog => todo |
2015-07-07 22:23 | Alan Scott | Note Added: 0034686 | |
2015-07-08 16:30 | Utkarsh Ayachit | Status | todo => backlog |
2015-07-08 16:36 | Utkarsh Ayachit | Status | backlog => todo |
2015-07-14 13:20 | David C. Lonie | Note Added: 0034775 | |
2015-07-14 13:20 | David C. Lonie | Status | todo => active development |
2015-07-14 13:20 | David C. Lonie | Status | active development => gatekeeper review |
2015-07-14 13:20 | David C. Lonie | Resolution | open => fixed |
2015-07-27 11:27 | Utkarsh Ayachit | Note Added: 0034825 | |
2015-07-27 11:27 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2015-07-27 11:27 | Utkarsh Ayachit | Fixed in Version | => git-master |
2015-07-27 18:11 | Alan Scott | Note Added: 0034828 | |
2015-07-27 18:11 | Alan Scott | Status | customer review => todo |
2015-07-28 09:59 | David C. Lonie | Note Added: 0034845 | |
2015-07-28 10:00 | David C. Lonie | Assigned To | David C. Lonie => Utkarsh Ayachit |
2015-08-10 11:11 | Utkarsh Ayachit | Assigned To | Utkarsh Ayachit => Cory Quammen |
2015-08-10 11:12 | Utkarsh Ayachit | Note Added: 0034937 | |
2015-08-10 18:31 | Utkarsh Ayachit | Assigned To | Cory Quammen => David C. Lonie |
2015-08-11 08:53 | David C. Lonie | Note Added: 0034938 | |
2015-08-11 11:15 | Utkarsh Ayachit | Relationship added | related to 0015629 |
2015-08-11 11:15 | Utkarsh Ayachit | Note Added: 0034940 | |
2015-08-11 11:15 | Utkarsh Ayachit | Status | todo => customer review |
2015-08-11 18:02 | Alan Scott | Note Added: 0034962 | |
2015-08-11 18:02 | Alan Scott | Status | customer review => todo |
2015-08-13 16:58 | Cory Quammen | Note Added: 0034993 | |
2015-08-14 10:54 | Cory Quammen | Assigned To | David C. Lonie => Cory Quammen |
2015-08-14 10:54 | Cory Quammen | Status | todo => active development |
2015-08-15 09:31 | Cory Quammen | Topic Name | => 13130_fix_scalar_bar_settings |
2015-08-15 09:31 | Cory Quammen | Note Added: 0034998 | |
2015-08-15 09:31 | Cory Quammen | Status | active development => gatekeeper review |
2015-08-15 09:31 | Cory Quammen | Fixed in Version | git-master => 4.4 |
2015-08-18 23:49 | Cory Quammen | Note Added: 0035025 | |
2015-08-19 08:09 | Utkarsh Ayachit | Note Added: 0035027 | |
2015-08-19 08:09 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2015-09-02 16:53 | Alan Scott | Note Added: 0035137 | |
2015-09-02 16:53 | Alan Scott | Status | customer review => closed |
2015-09-03 05:41 | Mathieu Westphal | Relationship added | has duplicate 0015434 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |