bugGNU Octave - Bugs: bug #39326, initial "tightinset" is...

 
 

bug #39326: initial "tightinset" is wrong

Submitted by:  Ben Abbott <bpabbott>
Submitted on:  Sun 23 Jun 2013 11:14:05 AM UTC  
 
Category: Plotting with OpenGLSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Inaccurate Result
Status: FixedAssigned to: Ben Abbott <bpabbott>
Originator Name: Ben AbbottOpen/Closed: Closed
Release: devOperating System: Any

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Sun 28 Jul 2013 01:43:04 AM UTC, comment #6:

I pushed a changeset

http://hg.savannah.gnu.org/hgweb/octave/rev/786bcce466e9

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Sat 27 Jul 2013 04:40:19 PM UTC, comment #5:

I've attached a changeset that handles the initializes the tightinset before children are added to the axes.

(file #28673)

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Sat 27 Jul 2013 12:43:14 PM UTC, comment #4:

There remains a small bug. The tightinset() is not updated when the axes are first created.

A start would be to add a call to sync_positions() to axes::update_font().

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Thu 27 Jun 2013 12:54:53 AM UTC, comment #3:

I'm looking at another changeset to simplify subplot.m (reduce the special gnuplot stuff). I'l wait on that until this is resolved.

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Wed 26 Jun 2013 11:21:04 PM UTC, comment #2:

Ben,

I sent you an email about the fact that this changeset
introduces a regression with respect to subplots.
I looked at the changeset, but could not find where it
causes the size of a subplot to go wrong.

Anyhow, if I rollback this changeset the problem goes away.

I hope you can find the problem since this changeset also
does a lot of good!

Michael

Michael Godfrey <godfrey>
Mon 24 Jun 2013 06:58:22 AM UTC, comment #1:

I pushed a changeset

http://hg.savannah.gnu.org/hgweb/octave/rev/e528beb2d774

Ben Abbott <bpabbott>
Project MemberIn charge of this item.
Sun 23 Jun 2013 11:14:05 AM UTC, original submission:

The frontend doesn't initialize the "tightinset" correctly.

Running the script above, produces the result below.

The tight inset should be set equal to the difference between the axes bounding box (including the title, {x,y,z}labels and {x,y,z}ticklabels) and the axes "position" (which is just the bounding box for the axes' plot area). See the link below for more information.

http://www.mathworks.com/help/matlab/creating_plots/automatic-axes-resize.html

The "looseinset" is an undocumented Matlab property. There is some info on it from non-mathworks sources.

http://undocumentedmatlab.com/blog/axes-looseinset-property/

(Note: This bug/feature prevents legend() from consistently rendering a proper result)

Ben Abbott <bpabbott>
Project MemberIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #28673:  changeset.patch added by bpabbott (1kB - application/octet-stream)

 

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 bpabbott (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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 28 Jul 2013 01:43:04 AM UTCbpabbottStatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Sat 27 Jul 2013 04:40:19 PM UTCbpabbottAttached File-=>Added changeset.patch, #28673
      StatusReady For Test=>Patch Submitted
    Mon 24 Jun 2013 06:58:22 AM UTCbpabbottStatusNone=>Ready For Test
      Assigned toNone=>bpabbott

    Back to the top


    Powered by Savane 3.1-cleanup