View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015074ParaView(No Category)public2014-10-24 15:152016-08-12 09:59
ReporterAlan Scott 
Assigned ToCory Quammen 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionmoved 
PlatformOSOS Version
Product Version4.2 
Target Version5.1Fixed in Version 
Summary0015074: Maximum number of labels (Color Legend) is wrong
DescriptionThe maximum number of labels for the Color Legend is incorrect, from a user's perspective. The maximum number of labels on the right side of the color legend should equal the maximum number of labels. This includes the min and max label. (Users really don't care that min and max aren't minor labels - they just count up the number of labels, and wonder why there are more than the max).

Also, please initialize it to 7 - i.e., same as now (5 minor plus min and max).

I couldn't find where this count is initialized, but the bulk of the code is in vtkPVScalarBarActor::ConfigureTicks() (and sub funcitons). It is called NumberOfLabels.

TagsNo tags attached.
ProjectSandia
Topic Name
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0033665)
Alan Scott (manager)
2014-10-24 15:16

 Color legend cleanup.
(0033729)
Utkarsh Ayachit (administrator)
2014-11-05 11:49

 This is really not easy to do the way things stand. I am going to delay this for later. Let's plan revisiting color legend and cube axes entirely in near future. The way things stand, both these could use some major cleanups/refactoring.
(0034647)
David C. Lonie (developer)
2015-07-02 11:03

 Just to add: the "maximum number of labels" option disappears with the new vtkAxis-based placement patch, and the number of labels is determined automatically using the same algorithm as the 2D chart axes.
(0034648)
David C. Lonie (developer)
2015-07-02 11:03

 Should we close this one, since it's no longer relevant with the new implementation?
(0034652)
Utkarsh Ayachit (administrator)
2015-07-02 11:12

 We will close with appropriate comments once the change lands. It hasn't, right?
(0034653)
David C. Lonie (developer)
2015-07-02 11:24

 Nope, it's not merged yet, but here's the PR for reference: https://gitlab.kitware.com/paraview/paraview/merge_requests/183 [^]

Sounds good to me.
(0035724)
Alan Scott (manager)
2016-02-11 22:19

 Has this one landed? Can we move to customer review?
(0035728)
David C. Lonie (developer)
2016-02-12 10:17

 I don't actually know what happened with this. Either Utkarsh or Cory took over the scalar bar stuff around the time that merge request was made. Let's see what they say.
(0035735)
Utkarsh Ayachit (administrator)
2016-02-13 11:41

 Cory, can you check if this is still in your implementation?
(0035736)
Cory Quammen (developer)
2016-02-14 23:40

 Yes, it should work fine. I just verified.
(0038692)
Kitware Robot (administrator)
2016-08-12 09:59

 Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current ParaView Issues page linked in the banner at the top of this page.

 Issue History
Date Modified Username Field Change
2014-10-24 15:15 Alan Scott New Issue
2014-10-24 15:16 Alan Scott Note Added: 0033665
2014-10-24 15:16 Alan Scott Target Version => 4.3
2014-11-04 10:02 Utkarsh Ayachit Target Version 4.3 => bug_fix
2014-11-05 11:49 Utkarsh Ayachit Note Added: 0033729
2014-11-05 11:49 Utkarsh Ayachit Target Version bug_fix => 4.3
2014-11-14 22:52 Utkarsh Ayachit Target Version 4.3 => 4.4
2015-07-02 10:49 Utkarsh Ayachit Assigned To => David C. Lonie
2015-07-02 11:03 David C. Lonie Note Added: 0034647
2015-07-02 11:03 David C. Lonie Note Added: 0034648
2015-07-02 11:12 Utkarsh Ayachit Note Added: 0034652
2015-07-02 11:24 David C. Lonie Note Added: 0034653
2015-07-08 16:36 Utkarsh Ayachit Status backlog => todo
2015-09-11 16:43 Utkarsh Ayachit Target Version 4.4 => 5.1
2016-02-11 22:19 Alan Scott Note Added: 0035724
2016-02-12 10:17 David C. Lonie Note Added: 0035728
2016-02-12 10:17 David C. Lonie Assigned To David C. Lonie => Utkarsh Ayachit
2016-02-13 11:40 Utkarsh Ayachit Assigned To Utkarsh Ayachit => Cory Quammen
2016-02-13 11:41 Utkarsh Ayachit Note Added: 0035735
2016-02-14 23:40 Cory Quammen Note Added: 0035736
2016-03-03 11:13 Utkarsh Ayachit Status todo => backlog
2016-08-12 09:59 Kitware Robot Note Added: 0038692
2016-08-12 09:59 Kitware Robot Status backlog => closed
2016-08-12 09:59 Kitware Robot Resolution open => moved

Copyright © 2000 - 2018 MantisBT Team