bugGNU Octave - Bugs: bug #39359, printing plot with legend modifies...

 
 

bug #39359: printing plot with legend modifies position

Submitter:  Ben Abbott <bpabbott>
Submitted:  Fri 28 Jun 2013 03:15:55 PM UTC
   
 
Category:  Plotting with gnuplot Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  bpabbott
Originator Name:  Ben Abbott Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Fri 19 Jul 2013 02:35:54 PM UTC, comment #7: 

I'm closing this report.  There are additional changes that are needed to clean up legend.m.  I hope to get to that in the next few weeks.

Ben Abbott <bpabbott>
Group Member
Mon 01 Jul 2013 08:47:39 AM UTC, comment #6: 

Thanks.  I added an unwind_protect block to the test and pushed the changeset

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

Ben Abbott <bpabbott>
Group Member
Mon 01 Jul 2013 08:37:01 AM UTC, comment #5: 

Your patch fixes both errors I reported in bug report
  #39321 (print(..., '-Ffont')) and
  #39327 (shrinking axes)

Stefan Mahr <dac922>
Mon 01 Jul 2013 08:24:28 AM UTC, comment #4: 

Updated patch attached (fixed typos in the test and created a hg changeset)

(file #28465)

Ben Abbott <bpabbott>
Group Member
Mon 01 Jul 2013 07:50:25 AM UTC, comment #3: 

Opps.  Its not "string" that has a problem/bug, but "interpreter".

Ben Abbott <bpabbott>
Group Member
Mon 01 Jul 2013 07:30:51 AM UTC, comment #2: 

Stefan,

I found a trivial solution to the shrinking axes.  A diff is attached.

This fixes the print problem but the legend box and line spacing no longer adjust to a change in fontsize (or change in string, which was already a bug).  If this fixes all other problems you've seen, I'll push it and work on fixing the legend box.

(file #28464)

Ben Abbott <bpabbott>
Group Member
Fri 28 Jun 2013 03:26:43 PM UTC, comment #1: 

Please try this instead:

hg revert -r d749c9b588e5 scripts/plot/legend.m


Stefan Mahr <dac922>
Fri 28 Jun 2013 03:15:55 PM UTC, original submission:  

Repeating the command below (id 7b53de2c7412) results in a shrinking figure axes size.


plot (1:2); legend ("a"); print test.png


Its been reported that the problem does not exist prior to 286904321282.  However, I'm unable to confirm.  I ran these commands;


hg revert -a -r d749c9b588e5
./run-octave --no-gui
close all
graphics_toolkit gnuplot
demo legend 1
print test.png


And encountered a bug with printing.


error: set: invalid handle (= -19.1813)
error: set: invalid handle (= -19.1813)
error: called from:
error:   /Users/bpabbott/Development/mercurial/default/sources/scripts/plot/private/__gnuplot_print__.m at line 177, column 5
error:   /Users/bpabbott/Development/mercurial/default/sources/scripts/plot/private/__gnuplot_print__.m at line 123, column 5
error:   /Users/bpabbott/Development/mercurial/default/sources/scripts/plot/print.m at line 414, column 12


If I revert further,


hg revert -a -r cddf9103a566


Then all works as it should.

Ben Abbott <bpabbott>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #28465:  changeset.patch added by bpabbott (1KiB - application/octet-stream)
file #28464:  legend.diff added by bpabbott (956B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dac922 (Posted a comment)
  • -email is unavailable- added by bpabbott (Submitted the item)
  • -email is unavailable- added by bpabbott (Stefan, I've started a new bug report. Please add your comments here)
  •  

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

    Date Changed by Updated Field Previous Value => Replaced by
    2013-07-20 bpabbott Open/ClosedOpen Closed
    2013-07-19 bpabbott StatusReady For Test Fixed
    2013-07-01 bpabbott StatusNone Ready For Test
    2013-07-01 bpabbott Attached File- Added changeset.patch, #28465
    2013-07-01 bpabbott Attached File- Added legend.diff, #28464
    2013-06-28 bpabbott Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code