bugGNU Octave - Bugs: bug #43543, [octave forge] (control) rlocus...

 
 

bug #43543: [octave forge] (control) rlocus unnecesary figure manipulation

Submitter:  None
Submitted:  Wed 05 Nov 2014 06:23:01 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Performance
Status:  Wont Fix Assigned to:  None
Originator Name:  Alberto Hamilton Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * 3.8.1
Operating System:  * GNU/Linux Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 06 Nov 2014 07:41:37 PM UTC, comment #2: 

These lines were introduced in control 2.6.3 in order to support the FLTK graphics backend. If you have a better workaround, please let me know.

Lukas Reichlin <paramaniac>
Thu 06 Nov 2014 12:41:32 PM UTC, comment #1: 

Adding maintainer of the control package to the CC list of this bug report.

However, note that there are plenty of functions that create figures when called with no output arguments, most of the bw* functions in the image package for example (this behaviour is required for Matlab compatibility). I think the logic here is that if you don't request an output argument, you are probably using it as an interactive tool for exploration, in which case creating a figure is what you want.

Why are you calling the function in a non-interactive invocation, if you are not storing the output in a variable?


Carnë Draug <carandraug>
Group Member
Wed 05 Nov 2014 06:23:01 PM UTC, original submission:  

On octave control pkg (vesion control-2.6.4).

rlocus function, if no output arguments are requested, plots the Root Locus diagram. To do that it:
- invoke 'plot' (line 288)
- set the current figure to not visible (line 289)
- invoke 'plot' again (line 290)
- modify certain parameters
- makes the figure visible again (line 304)

I think that this procedure in unnecesary and interfere whith non interactive invocations to Octave (used directly to print graphics results)

So lines 288, 289 and 304 should be removed.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mtmiller (Updated the item)
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by paramaniac (Posted a comment)
  • -email is unavailable- added by carandraug (Lukas Reichlin - maintainer of control package)
  • -email is unavailable- added by None (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
    2017-08-14 mtmiller StatusNone Wont Fix
        Open/ClosedOpen Closed
        Summary[octave forge] (rlocus) unnecesary figure manipulation [octave forge] (control) rlocus unnecesary figure manipulation
    2017-08-13 jwe Summaryrlocus unnecesary figure manipulation [octave forge] (rlocus) unnecesary figure manipulation
    2014-11-06 carandraug Carbon-Copy- Added -email is unavailable-
    2014-11-06 rik5 Carbon-CopyRemoved 72865 -
    2014-11-06 rik5 CategoryLibraries Octave Package

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code