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

 
 

bug #39326: initial "tightinset" is wrong

Submitted by:  Ben Abbott <bpabbott>
Submitted on:  Sun Jun 23 11:14:05 2013  
 
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 Jul 28 01:43:04 2013, 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 Jul 27 16:40:19 2013, 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 Jul 27 12:43:14 2013, 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 Jun 27 00:54:53 2013, 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 Jun 26 23:21:04 2013, 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 Jun 24 06:58:22 2013, 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 Jun 23 11:14:05 2013, 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 Jul 28 01:43:04 2013bpabbottStatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Sat Jul 27 16:40:19 2013bpabbottAttached File-=>Added changeset.patch, #28673
      StatusReady For Test=>Patch Submitted
    Mon Jun 24 06:58:22 2013bpabbottStatusNone=>Ready For Test
      Assigned toNone=>bpabbott

    Back to the top


    Powered by Savane 3.1-cleanup