bugGNU Octave - Bugs: bug #41396, existing figure is made visible...

 
 

bug #41396: existing figure is made visible even when property is set to 'off'

Submitter:  Markus Appel <mappel>
Submitted:  Wed 29 Jan 2014 08:16:28 PM UTC
   
 
Category:  Plotting Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 3.8.0 Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 30 Jan 2014 04:27:48 PM UTC, comment #3: 

I applied your patch to the stable branch (http://hg.savannah.gnu.org/hgweb/octave/rev/0f32b34f6ec0) ahead of the 3.8.1 release.

Rik <rik5>
Group administrator
Thu 30 Jan 2014 01:06:06 AM UTC, comment #2: 

I see that part of the problem is that I tried to get us closer to Matlab compatibility (http://www.mathworks.com/help/matlab/ref/figure.html), but they don't support the syntax


figure (N, 'property', value)


This is convenient for them because they don't have this problem.

I'll review your patch soon.

Rik <rik5>
Group administrator
Wed 29 Jan 2014 08:31:04 PM UTC, comment #1: 

Patch attached. Make figure N visible unless explicitly asked not to.

(file #30418)

Markus Appel <mappel>
Wed 29 Jan 2014 08:16:28 PM UTC, original submission:  

Some of my old plotting scripts crashed in 3.8 when executed via remote shell without window system, because figure(1,'visible','off') tries to open a figure window when the figure is already existent:


octave:1> figure(1,'visible','off')
% No window opened
octave:2> get(gcf,'visible')
ans = off

octave:3> figure(1,'visible','off')
% Existing figure 1 now made visible despite explicit property setting
octave:4> get(gcf,'visible')
ans = on


Seems like this was introduced in

# HG changeset patch
# User Rik <rik@octave.org>
# Date 1375381132 25200
# Node ID 603ce503babd69b55bf9eecbb9386cc1f4fa0fc2
# Parent  e4968b4613a52c8219af7b9d4cd2826f26afdcfe
figure.m: Overhaul function.  When switching to figure N make it visible.


Markus Appel <mappel>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #30418:  octave-fix-figure.patch added by mappel (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by mappel (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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-01-30 rik5 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2014-01-30 rik5 StatusNone Patch Submitted
    2014-01-29 mappel Attached File- Added octave-fix-figure.patch, #30418

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code