bugGNU Octave - Bugs: bug #51732, plotyy does not adjust position...

 
 

bug #51732: plotyy does not adjust position based on title size (multi-line title or large font size)

Submitter:  Dmitri A. Sergatskov <dasergatskov>
Submitted:  Fri 11 Aug 2017 05:23:18 PM UTC
   
 
Category:  Plotting Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * other Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sat 23 Nov 2019 06:58:46 PM UTC, comment #8: 

All fixed now.  Closing report.

Rik <rik5>
Group administrator
Sat 23 Nov 2019 02:35:29 PM UTC, comment #7: 

Looks like this is fixed by the recent changes to plotyy.m as part of legend overhaul.

Pantxo Diribarne <pantxo>
Group Member
Tue 06 Mar 2018 05:03:24 PM UTC, comment #6: 

@Pantxo: This definitely starts to resolve the problem.  It seems that there may need to be a listener somewhere as well.  Try this code


close all
demo plotyy 1
title ({"1","2","3","4"})
title ("1")


This ends up leaving a gap at the top of the plot.


Rik <rik5>
Group administrator
Tue 06 Mar 2018 04:21:51 PM UTC, comment #5: 

Resetting title of report.

Rik <rik5>
Group administrator
Tue 06 Mar 2018 04:21:35 PM UTC, comment #4: 

I attached a second patch that sets outerposition related properties all at once.

(file #43477)

Pantxo Diribarne <pantxo>
Group Member
Tue 06 Mar 2018 04:12:42 PM UTC, comment #3: 

I attached a patch that solves this bug for me.

(file #43476)

Pantxo Diribarne <pantxo>
Group Member
Tue 06 Mar 2018 04:06:54 PM UTC, comment #2: 

This is an example from Dmitri showing a large font size is enough to screw up positioning.


set(0, 'DefaultAxesFontsize', 24)
x=0:10;
y1 = randn(1,11);
y2 = randn(1,11);
plotyy(x,y1, x,y2);
title ("Title");


And see the attached big_title.png.


Rik <rik5>
Group administrator
Fri 22 Dec 2017 04:17:52 PM UTC, comment #1: 

It's true.  I just re-coded a lot of plotyy and the positioning algorithm does not take in to account xlabels, ylabels, or title.

Rik <rik5>
Group administrator
Fri 11 Aug 2017 05:23:18 PM UTC, original submission:  

See "demo plotyy 1". Attached is the result with qt backend on
bd9e719f04cc tip @; but it also present on 4.2.1 and any other backend.

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 #43477:  update_outerposition2.patch added by pantxo (2KiB - text/x-patch)
file #43476:  update_outerposition.patch added by pantxo (2KiB - text/x-patch)
file #43475:  big_title.png added by rik5 (34KiB - image/png)
file #41486:  bad_plotyy.png added by dasergatskov (29KiB - 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 pantxo (Updated the item)
  • -email is unavailable- added by rik5 (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 12 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-11-23 rik5 Open/ClosedOpen Closed
    2019-11-23 rik5 StatusReady For Test Fixed
    2019-11-23 pantxo StatusPatch Submitted Ready For Test
    2018-03-06 rik5 Summaryplotyy does not accomodate multi-line title plotyy does not adjust position based on title size (multi-line title or large font size)
    2018-03-06 pantxo Attached File- Added update_outerposition2.patch, #43477
    2018-03-06 pantxo Attached File- Added update_outerposition.patch, #43476
        StatusConfirmed Patch Submitted
        Summaryplotyy does not adjust position based on title size (multi-line title or large font size) plotyy does not accomodate multi-line title
    2018-03-06 rik5 Attached File- Added big_title.png, #43475
        Summaryplotyy does not accomodate multi-line title plotyy does not adjust position based on title size (multi-line title or large font size)
    2017-12-22 rik5 StatusNone Confirmed
    2017-08-11 dasergatskov Attached File- Added bad_plotyy.png, #41486

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code