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

 
 

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

Submitted by:  Markus Appel <mappel>
Submitted on:  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

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

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>
Project 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>
Project 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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project 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.5