bugGNU Octave - Bugs: bug #54937, plotting fails with -W...

 
 

bug #54937: plotting fails with -W (--no-window-system)

Submitter:  Colin Macdonald <cbm>
Submitted:  Fri 02 Nov 2018 05:36:59 AM UTC
   
 
Category:  Plotting with gnuplot Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  pantxo
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 15 Nov 2018 09:59:14 PM UTC, comment #5: 

Fixed the spelling of "disallow" and checked in the changeset here: https://hg.savannah.gnu.org/hgweb/octave/rev/32d4ee071348.

Everything works for me now, closing report.

Rik <rik5>
Group administrator
Thu 15 Nov 2018 08:33:57 PM UTC, comment #4: 

In cset ae9d6a491c06 I changed Fdrawnow so that it allows recursion (needed to have callbacks recursively interruptible). The attached patch restores the old behavior when there is no display.

(file #45436)

Pantxo Diribarne <pantxo>
Group Member
Thu 15 Nov 2018 01:11:10 AM UTC, comment #3: 

Also adding Pantxo to the CC list.

Rik <rik5>
Group administrator
Thu 15 Nov 2018 01:10:26 AM UTC, comment #2: 

Using hg bisect the changeset that introduced this was


changeset:   25960:ae9d6a491c06
user:        Pantxo Diribarne <pantxo.diribarne@gmail.com>
date:        Wed Aug 22 09:14:44 2018 +0200
summary:     Refactor graphics event handling and fix interruption of callbacks (bug #54340)


I'm adding Pantxo to this bug report.

My guess is that drawnow() is being called from within the C++ code during the cronstruction of a graphics object.  A simpler test example is


run-octave -f -q -W
figure ()



Rik <rik5>
Group administrator
Fri 02 Nov 2018 05:23:25 PM UTC, comment #1: 

Confirmed.

There is lots of screwy stuff going on here.  Even trying to debug it led to strange things where the keyboard() function did not stop the interpreter.


Rik <rik5>
Group administrator
Fri 02 Nov 2018 05:36:59 AM UTC, original submission:  

This is with "hg id: 38a881b8fbec".

If I start with "--no-window-system", Octave used to use some ascii art (via gnuplot I believe).

Now I get:

>> t = linspace(0, 1, 10);
>> plot(t,sin(t))
error: max_recursion_depth exceeded
error: called from
    __gnuplot_drawnow__ at line 91 column 9


In Octave 4.4.2, I get:

octave:1> t = linspace(0, 1, 10);
octave:2> plot(t,sin(t))

   1 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
     >        +       +        +       +        >
     >                                        ***
 0.8 >-+                                 *****+->
     >                              *****       >
 0.6 >-+                        ****          +->
     >                     *****                >
     >                 ****                     >
 0.4 >-+             **                       +->
     >           ****                           >
 0.2 >-+      ***                             +->
     >      **                                  >
     >  ****  +       +        +       +        >
   0 ***>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
     0       0.2     0.4      0.6     0.8       1


Probably no one really uses these sorts of things anymore but they are useful in automated testing.

Downstream bug: https://github.com/cbm755/octsympy/issues/915

I have not bisected this yet but it must be somewhat recent (last couple months), maybe
http://hg.savannah.gnu.org/hgweb/octave/rev/811e094ba676


Colin Macdonald <cbm>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #45436:  nodisplay.patch added by pantxo (2KiB - 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 pantxo (Updated the item)
  • -email is unavailable- added by rik5
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by cbm (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 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-11-15 rik5 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2018-11-15 pantxo Attached File- Added nodisplay.patch, #45436
        StatusConfirmed Patch Submitted
    2018-11-15 rik5 Carbon-Copy- Added pantxo
    2018-11-15 rik5 Assigned toNone pantxo
    2018-11-02 rik5 StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code