* Bug #13743 fixed - neldermead_configure failed to set options '-gamma' and '-sigma'. 61/18561/3
Adeline CARNIS [Wed, 28 Sep 2016 10:28:01 +0000 (12:28 +0200)]
Change-Id: Ie169e27643bba471249b3a7edc17caa88a4dc07f

scilab/CHANGES.md
scilab/modules/optimization/macros/neldermead/neldermead_configure.sci
scilab/modules/optimization/tests/nonreg_tests/bug_13743.dia.ref [new file with mode: 0644]
scilab/modules/optimization/tests/nonreg_tests/bug_13743.tst [new file with mode: 0644]

index 4f6ff45..c75b4f2 100644 (file)
@@ -329,6 +329,7 @@ Bug Fixes
 * [#13470](http://bugzilla.scilab.org/show_bug.cgi?id=13470): `histplot(0,0,%t)` crashed
 * [#13583](http://bugzilla.scilab.org/show_bug.cgi?id=13583): `getd` loading a script including a `clear` instruction yielded an error
 * [#13597](http://bugzilla.scilab.org/show_bug.cgi?id=13597): `help format` claimed setting a number of digits instead of characters
+* [#13743](http://bugzilla.scilab.org/show_bug.cgi?id=13743): `neldermead_configure` failed to set options "-gamma" and "-sigma".
 * [#13757](http://bugzilla.scilab.org/show_bug.cgi?id=13757): The toolbox menu dit not load properly ATOMS modules not autoloaded.
 * [#13990](http://bugzilla.scilab.org/show_bug.cgi?id=13990): `warning` with localization enabled some memory corruption
 * [#14192](http://bugzilla.scilab.org/show_bug.cgi?id=14192): `g_margin` error-ed for double integrator.
index bf02cc0..538f9ed 100644 (file)
@@ -62,12 +62,12 @@ function this = neldermead_configure (this,key,value)
     case "-gamma" then
         nelmead_typereal ( value , "value" , 3 );
         nelmead_checkscalar ( "neldermead_configure" , value , "value" , 3 )
-        nelmead_checkrange ( "neldermead_configure" , var , "value" , 3 , UN , 1-%eps )
+        nelmead_checkrange ( "neldermead_configure" , value , "value" , 3 , UN , 1-%eps )
         this.gamma = value;
     case "-sigma" then
         nelmead_typereal ( value , "value" , 3 );
         nelmead_checkscalar ( "neldermead_configure" , value , "value" , 3 )
-        nelmead_checkrange ( "neldermead_configure" , var , "value" , 3 , UN , 1-%eps )
+        nelmead_checkrange ( "neldermead_configure" , value , "value" , 3 , UN , 1-%eps )
         this.sigma = value;
     case "-tolsimplexizeabsolute" then
         nelmead_typereal ( value , "value" , 3 );
diff --git a/scilab/modules/optimization/tests/nonreg_tests/bug_13743.dia.ref b/scilab/modules/optimization/tests/nonreg_tests/bug_13743.dia.ref
new file mode 100644 (file)
index 0000000..85a83ca
--- /dev/null
@@ -0,0 +1,21 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2016 - Scilab Enterprises - Adeline CARNIS
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- CLI SHELL MODE -->
+//
+// <-- Non-regression test for bug 13743 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=13743
+//
+// <-- Short Description -->
+// neldermead_configure function failed to set options "-gamma" and "-sigma"
+nm = neldermead_new();
+ierr = execstr("nm = neldermead_configure(nm, ""-sigma"", 0.25)", "errcatch");
+assert_checkequal(ierr, 0);
+ierr = execstr("nm = neldermead_configure(nm, ""-gamma"", 0.25)", "errcatch");
+assert_checkequal(ierr, 0);
+nm = neldermead_destroy(nm);
diff --git a/scilab/modules/optimization/tests/nonreg_tests/bug_13743.tst b/scilab/modules/optimization/tests/nonreg_tests/bug_13743.tst
new file mode 100644 (file)
index 0000000..9ca8e28
--- /dev/null
@@ -0,0 +1,22 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2016 - Scilab Enterprises - Adeline CARNIS
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- CLI SHELL MODE -->
+//
+// <-- Non-regression test for bug 13743 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=13743
+//
+// <-- Short Description -->
+// neldermead_configure function failed to set options "-gamma" and "-sigma"
+
+nm = neldermead_new();
+ierr = execstr("nm = neldermead_configure(nm, ""-sigma"", 0.25)", "errcatch");
+assert_checkequal(ierr, 0);
+ierr = execstr("nm = neldermead_configure(nm, ""-gamma"", 0.25)", "errcatch");
+assert_checkequal(ierr, 0);
+nm = neldermead_destroy(nm);