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

 
 

bug #52302: axes color wrong in plotyy demo 3

Submitted by:  Dmitri A. Sergatskov <dasergatskov>
Submitted on:  Sat 28 Oct 2017 11:31:21 PM UTC  
 
Category: PlottingSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project Administrator
Tue 31 Oct 2017 09:59:20 AM UTC, comment #4:

None.

Michael Godfrey <godfrey>
Project 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>
Project 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

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 File(s):
   
   
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
  • -unavailable- added by godfrey (Posted a comment)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by sebald (Posted a comment)
  • -unavailable- added by dasergatskov (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 01 Nov 2017 09:06:39 PM UTCrik5StatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Sat 28 Oct 2017 11:31:21 PM UTCdasergatskovAttached File-=>Added t1.png, #42287

    Back to the top


    Powered by Savane 3.1-cleanup1