bugGNU Octave - Bugs: bug #52302, axes color wrong in plotyy demo 3

 
 

bug #52302: axes color wrong in plotyy demo 3

Submitter:  Dmitri A. Sergatskov <dasergatskov>
Submitted:  Sat 28 Oct 2017 11:31:21 PM UTC
   
 
Category:  Plotting Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 01 Nov 2017 09:06:39 PM UTC, comment #5: 

I deleted the unuseful demo here http://hg.savannah.gnu.org/hgweb/octave/rev/d5c38fdf77f9.  Marking as fixed and closing report.

Rik <rik5>
Group administrator
Tue 31 Oct 2017 09:59:20 AM UTC, comment #4: 

None.

Michael Godfrey <godfrey>
Group Member
Tue 31 Oct 2017 04:52:07 AM UTC, comment #3: 

No objection, I agree.  I'm trying to think of a scenario where the color advancement by two (due to two plots per command) would come into play, but can't imagine one.

Dan Sebald <sebald>
Tue 31 Oct 2017 04:27:15 AM UTC, comment #2: 

This demo probably should just be removed.  I think we may have added it to test Matlab conformance.  But it's really kind of a dumb thing to have multiple axes stepping all over each other.

Any objection to removing?

Rik <rik5>
Group administrator
Mon 30 Oct 2017 12:21:55 AM UTC, comment #1: 

What do you consider the wrong color?  I ask because there are a several observations.

First, the demo is a conflict in the sense of two y axes per plot, plotted one on top of the other means the axes coincide.  Hence one axis must cover the other.  That's why although the axes say

(orange) Left axis is Blue
(gold) Right axis is Orange

the color doesn't match.  (Take out the second plotyy, and the description is correct.)  The blue (cyan technically), left axis is actually "underneath" the orange, left axis.  But even "underneath" isn't correct as I tried the following to shift the range of the two different plots


%! [~, h3, h4] = plotyy (t+1, 2*x, t+1, 0.5*y);


but saw no remnants of blue alongside the orange.  So, the most logical change should really be "Left axis is Orange" and "Right axis is Gold".

But notice, the demo doesn't actually specify the color anywhere.  Someone must have just looked at the color and written the text.  Whether the color sequence scheme is guaranteed as versions go forward, who knows?

And one last observation, which may be more critical, is that the first plot was

Blue (left)  Orange (right)

and the second plot was

Orange (left)  Gold (right)

such that the color Orange is shared across two different plots, when maybe it should be

Gold (left)  Magenta (right)

or whatever color comes after Gold, for the second plot.  I suppose the plotting routines are advancing the color by one with each new plot not realizing that two plots are created per each plotyy.

Dan Sebald <sebald>
Sat 28 Oct 2017 11:31:21 PM UTC, original submission:  

axes color wrong in plotyy demo 3

See attached.

hg id 8365e584ebd8 tip @

Dmitri.
--

Dmitri A. Sergatskov <dasergatskov>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #42287:  t1.png added by dasergatskov (40KiB - image/png)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by godfrey (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by sebald (Posted a comment)
  • -email is unavailable- added by dasergatskov (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-11-01 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2017-10-28 dasergatskov Attached File- Added t1.png, #42287

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code