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

 
 

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

Submitted by:  None
Submitted on:  Wed 05 Nov 2014 06:23:01 PM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Performance
Status: Wont FixAssigned to: None
Originator Name: Alberto HamiltonOriginator Email: -unavailable-
Open/Closed: ClosedRelease: 3.8.1
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 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>
Project 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 File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mtmiller (Updated the item)
  • -unavailable- added by jwe (Updated the item)
  • -unavailable- added by paramaniac (Posted a comment)
  • -unavailable- added by carandraug (Lukas Reichlin - maintainer of control package)
  • -unavailable- added by None (Submitted the item)
  •  

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

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 14 Aug 2017 04:00:40 PM UTCmtmillerStatusNone=>Wont Fix
      Open/ClosedOpen=>Closed
      Summary[octave forge] (rlocus) unnecesary figure manipulation=>[octave forge] (control) rlocus unnecesary figure manipulation
    Sun 13 Aug 2017 01:33:25 PM UTCjweSummaryrlocus unnecesary figure manipulation=>[octave forge] (rlocus) unnecesary figure manipulation
    Thu 06 Nov 2014 12:41:32 PM UTCcarandraugCarbon-Copy-=>Added -unavailable-
    Thu 06 Nov 2014 05:20:59 AM UTCrik5Carbon-CopyRemoved 72865=>-
    Thu 06 Nov 2014 05:20:47 AM UTCrik5CategoryLibraries=>Octave Forge Package

    Back to the top


    Powered by Savane 3.1-cleanup1