patchGNU Octave - Patches: patch #10200, wrong axes using stairs in subplot

 
 

patch #10200: wrong axes using stairs in subplot

Submitter:  nicodemi <nicodemi>
Submitted:  Wed 20 Apr 2022 07:12:05 AM UTC
   
 
Category:  Core : other Priority:  5 - Normal
Status:  Bug Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 27 Apr 2022 06:03:52 PM UTC, comment #3: 

Closing this in favor of bug #62373.

Markus Mützel <mmuetzel>
Group administrator
Thu 21 Apr 2022 05:10:57 PM UTC, comment #2: 

Re-reading bug #61945 (for which the behavior of `newplot` was changed), it's probably not correct to change the "currentaxes" here either. Instead, it would probably be better if `__next_line_color__.m` and Co. accepted an axes argument instead of always referring to `gca`...

Markus Mützel <mmuetzel>
Group administrator
Thu 21 Apr 2022 12:25:11 PM UTC, comment #1: 

Thanks for your bug report. (It should probably better be on the bug tracker. But I don't know how to move it from this patch tracker to there now.)

I can confirm this bug. It is probably caused by the fact that "newplot" changed the "currentaxes" in previous versions. But it no longer does so in Octave 7. The function "stairs" still relies on the old behavior.

The attached patch should fix that.

It is probably worth checking all functions that use "newplot" whether they still assume the old behavior, too.

(file #53125)

Markus Mützel <mmuetzel>
Group administrator
Wed 20 Apr 2022 07:12:05 AM UTC, original submission:  

stairs ignores first arg: axes handle

H1 = subplot(2,1,1);
H2 = subplot(2,1,2);
stairs(H1, 1:5,1:5, 'b')
stairs(H2, 1:5,1:5, 'r')

both draw commands goes into last axes handle

This does not happen with plot(H, ...)

nicodemi <nicodemi>

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mmuetzel (Updated the item)
  • -email is unavailable- added by nicodemi (Submitted the item)
  • -email is unavailable- added by nicodemi
  •  

    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 logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-04-27 mmuetzel StatusNone Bug
        Open/ClosedOpen Closed
        Dependencies- Depends on bugs #62373
    2022-04-21 mmuetzel Attached File- Added patch10200-stairs-currentaxes.patch, #53125
    2022-04-20 nicodemi Carbon-Copy- Added nicmus

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code