bugGNU Octave - Bugs: bug #38999, optimset("Disp",...

 
 

bug #38999: optimset("Disp", "off") not accepted

Submitted by:  Muhali <muhali>
Submitted on:  Wed 15 May 2013 02:20:30 PM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Tue 25 Jun 2013 09:57:55 AM UTC, comment #4:

So how would one do

without getting a warning? - You'd have to rename almost half of the optim-package options to become unique then.

Muhali <muhali>
Fri 24 May 2013 12:51:46 PM UTC, comment #3:

We warn for many things that Matlab does not. The code still runs and produces the desired result, doesn't it? You should clean up your code if it relies on behaviours such as division by zero or the braindead Matlab short-circuiting. It seems reasonable to me to also warn if you are using ambiguous options, since this might be an unintended error.

Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Fri 24 May 2013 07:10:09 AM UTC, comment #2:

Is the warning about ambiguous options actually necessary? It is different from ML and causes a lot of noise.

Muhali <muhali>
Fri 17 May 2013 04:58:08 PM UTC, comment #1:

Thanks for the patch. I checked in a slightly different solution using strncmpi. I also fixed optimget in a similar way and added some tests.

http://hg.savannah.gnu.org/hgweb/octave/rev/bc79ac595a05

John W. Eaton <jwe>
Project Administrator
Wed 15 May 2013 02:20:30 PM UTC, original submission:

octave warns and does not accept

while ML accepts "Disp" as long as it remains unambiguous (which I would prefer).

Here is a patch that fixes it (using the deprecated strmatch which I found easier to handle).

Muhali <muhali>

 

(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 jwe (Posted a comment)
  • -unavailable- added by jordigh (Updated the item)
  • -unavailable- added by muhali (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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 17 May 2013 04:58:08 PM UTCjweCategoryNone=>Octave Function
      StatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Wed 15 May 2013 02:44:57 PM UTCjordighStatusNone=>Patch Submitted
      Summaryoptimset("Disp", "off")=>optimset("Disp", "off") not accepted

    Back to the top


    Powered by Savane 3.1-cleanup