MantisBT - ParaView
View Issue Details
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 
ProjectSandia
Topic Name
Typeincorrect functionality
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.

Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-10-24 15:15Alan ScottNew Issue
2014-10-24 15:16Alan ScottNote Added: 0033665
2014-10-24 15:16Alan ScottTarget Version => 4.3
2014-11-04 10:02Utkarsh AyachitTarget Version4.3 => bug_fix
2014-11-05 11:49Utkarsh AyachitNote Added: 0033729
2014-11-05 11:49Utkarsh AyachitTarget Versionbug_fix => 4.3
2014-11-14 22:52Utkarsh AyachitTarget Version4.3 => 4.4
2015-07-02 10:49Utkarsh AyachitAssigned To => David C. Lonie
2015-07-02 11:03David C. LonieNote Added: 0034647
2015-07-02 11:03David C. LonieNote Added: 0034648
2015-07-02 11:12Utkarsh AyachitNote Added: 0034652
2015-07-02 11:24David C. LonieNote Added: 0034653
2015-07-08 16:36Utkarsh AyachitStatusbacklog => todo
2015-09-11 16:43Utkarsh AyachitTarget Version4.4 => 5.1
2016-02-11 22:19Alan ScottNote Added: 0035724
2016-02-12 10:17David C. LonieNote Added: 0035728
2016-02-12 10:17David C. LonieAssigned ToDavid C. Lonie => Utkarsh Ayachit
2016-02-13 11:40Utkarsh AyachitAssigned ToUtkarsh Ayachit => Cory Quammen
2016-02-13 11:41Utkarsh AyachitNote Added: 0035735
2016-02-14 23:40Cory QuammenNote Added: 0035736
2016-03-03 11:13Utkarsh AyachitStatustodo => backlog
2016-08-12 09:59Kitware RobotNote Added: 0038692
2016-08-12 09:59Kitware RobotStatusbacklog => closed
2016-08-12 09:59Kitware RobotResolutionopen => moved

Notes
(0033665)
Alan Scott   
2014-10-24 15:16   
Color legend cleanup.
(0033729)
Utkarsh Ayachit   
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   
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   
2015-07-02 11:03   
Should we close this one, since it's no longer relevant with the new implementation?
(0034652)
Utkarsh Ayachit   
2015-07-02 11:12   
We will close with appropriate comments once the change lands. It hasn't, right?
(0034653)
David C. Lonie   
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   
2016-02-11 22:19   
Has this one landed? Can we move to customer review?
(0035728)
David C. Lonie   
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   
2016-02-13 11:41   
Cory, can you check if this is still in your implementation?
(0035736)
Cory Quammen   
2016-02-14 23:40   
Yes, it should work fine. I just verified.
(0038692)
Kitware Robot   
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.