* Bug #13900 fixed - Statistics: making nanmin and nanmax obsolete 88/19188/6
Siddhartha Gairola [Tue, 7 Mar 2017 09:35:58 +0000 (14:35 +0530)]
Change-Id: I02e0446c03cf9ff149b42f11a7ee96982d9ce222

scilab/CHANGES.md
scilab/modules/slint/src/cpp/DeprecatedChecker.cpp
scilab/modules/statistics/help/en_US/data_missing_values/nanmax.xml
scilab/modules/statistics/help/en_US/data_missing_values/nanmin.xml
scilab/modules/statistics/help/ja_JP/data_missing_values/nanmax.xml
scilab/modules/statistics/help/ja_JP/data_missing_values/nanmin.xml
scilab/modules/statistics/macros/nanmax.sci
scilab/modules/statistics/macros/nanmin.sci
scilab/modules/statistics/tests/nonreg_tests/bug_1239.dia.ref [deleted file]
scilab/modules/statistics/tests/nonreg_tests/bug_1239.tst

index 9758287..ace3a8c 100644 (file)
@@ -303,6 +303,8 @@ Obsolete functions or features
 * `eval3d` will be removed from Scilab 6.1. Please use `ndgrid` instead.
 * `strcmpi` is deprecated. Please use `strcmp(..,"i")`instead.
 * `square` will be removed from Scilab 6.1. Please use `gcf().axes_size` and `replot` instead.
+* `nanmin` is obsolete and will be removed in Scilab 6.1. Please use `min` instead.
+* `nanmax` is obsolete and will be removed in Scilab 6.1. Please use `max` instead.
 
 
 Removed Functions
@@ -351,6 +353,7 @@ the [development mailing list](dev@lists.scilab.org) for a particular toolbox.
 * [#11756](http://bugzilla.scilab.org/show_bug.cgi?id=11756): In the categories of the ATOMS GUI, actually available modules might be not listed.
 * [#12771](http://bugzilla.scilab.org/show_bug.cgi?id=12771): xcosPalGenerateAllIcons help example was broken.
 * [#13592](http://bugzilla.scilab.org/show_bug.cgi?id=13592): In an axes in a uicontrol frame, setting a `legend` interactively might not follow the mouse accurately.
+* [#13900](http://bugzilla.scilab.org/show_bug.cgi?id=13900): `nanmin` and `nanmax` were useless duplicates of `min` and `max`
 * [#14254](http://bugzilla.scilab.org/show_bug.cgi?id=14254): When installing an ATOMS module that is not locally registered and available, the error message was unclear.
 * [#14376](http://bugzilla.scilab.org/show_bug.cgi?id=14376): input() is broken: \n introduced before prompting, multiple prompts, missing assignment, "%" "\n" "\t" no longer supported in messages...
 * [#14399](http://bugzilla.scilab.org/show_bug.cgi?id=14399): Whereami : wrong information (line numbers).
index 26f647b..7ff75df 100644 (file)
@@ -92,6 +92,13 @@ std::unordered_map<std::wstring, std::wstring> DeprecatedChecker::initDep()
 {
     // TODO: get this list from a conf file
     std::unordered_map<std::wstring, std::wstring> map;
+    // Scilab 6.0.0 => 6.1.0
+    map.emplace(L"square", L"replot");
+
+    // Scilab 6.0.0 => 6.0.1
+    map.emplace(L"nanmin", L"min");
+    map.emplace(L"nanmax", L"max");
+
     // Scilab 5.5.2 => 6.0.0
     map.emplace(L"fort", L"call");
     map.emplace(L"znaupd", L"eigs");
index f7c9167..f9e35d7 100644 (file)
@@ -1,22 +1,28 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!--
- * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
- * Copyright (C) 2000 - INRIA - Carlos Klimann
- *
- * Copyright (C) 2012 - 2016 - Scilab Enterprises
- *
- * This file is hereby licensed under the terms of the GNU GPL v2.0,
- * pursuant to article 5.3.4 of the CeCILL v.2.1.
- * This file was originally licensed under the terms of the CeCILL v2.1,
- * and continues to be available under such terms.
- * For more information, see the COPYING file which you should have received
- * along with this program.
- *
- -->
-<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org" xml:lang="en" xml:id="nanmax">
+* Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+* Copyright (C) 2000 - INRIA - Carlos Klimann
+*
+* Copyright (C) 2012 - 2016 - Scilab Enterprises
+*
+* This file is hereby licensed under the terms of the GNU GPL v2.0,
+* pursuant to article 5.3.4 of the CeCILL v.2.1.
+* This file was originally licensed under the terms of the CeCILL v2.1,
+* and continues to be available under such terms.
+* For more information, see the COPYING file which you should have received
+* along with this program.
+*
+-->
+<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink"
+          xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML"
+          xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org"
+          xml:lang="en" xml:id="nanmax">
     <refnamediv>
         <refname>nanmax</refname>
-        <refpurpose>max  (ignoring Nan's) </refpurpose>
+        <refpurpose>
+          max  (ignoring Nan's) <emphasis role="bold">This function is obsolete</emphasis>,
+          replaced with <link linkend="max">max</link>
+        </refpurpose>
     </refnamediv>
     <refsynopsisdiv>
         <title>Syntax</title>
     </refsection>
     <refsection>
         <title>Description</title>
+        <warning>
+          This function is obsolete and will  be removed from Scilab 6.1. Please use
+          <code>max()</code> instead.
+        </warning>
         <para>
             This function gives for a real or a numerical matrix
             <literal> x</literal> his largest  element <literal> m</literal> (but ignoring the
     <refsection>
         <title>Examples</title>
         <programlisting role="example"><![CDATA[
-x=[0.2113249 %nan 0.6653811;0.7560439 0.3303271 0.6283918]
-m=nanmax(x)
-m=nanmax(x,'r')
-m=nanmax(x,'c')
- ]]></programlisting>
+            x=[0.2113249 %nan 0.6653811;0.7560439 0.3303271 0.6283918]
+            m=nanmax(x)
+            m=nanmax(x,'r')
+            m=nanmax(x,'c')
+            ]]></programlisting>
     </refsection>
     <refsection>
         <title>Bibliography</title>
@@ -89,4 +99,16 @@ m=nanmax(x,'c')
             Wonacott, T.H. &amp; Wonacott, R.J.; Introductory Statistics, fifth edition, J.Wiley &amp; Sons, 1990.
         </para>
     </refsection>
+    <refsection>
+        <title>History</title>
+        <revhistory>
+            <revision>
+                <revnumber>6.0.1</revnumber>
+                <revremark>
+                    Function tagged as obsolete, to be removed from Scilab 6.1, replaced with
+                    <link linkend="max">max</link>.
+                </revremark>
+            </revision>
+        </revhistory>
+    </refsection>
 </refentry>
index ee40598..bf0279f 100644 (file)
@@ -1,22 +1,27 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!--
- * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
- * Copyright (C) 2000 - INRIA - Carlos Klimann
- *
- * Copyright (C) 2012 - 2016 - Scilab Enterprises
- *
- * This file is hereby licensed under the terms of the GNU GPL v2.0,
- * pursuant to article 5.3.4 of the CeCILL v.2.1.
- * This file was originally licensed under the terms of the CeCILL v2.1,
- * and continues to be available under such terms.
- * For more information, see the COPYING file which you should have received
- * along with this program.
- *
- -->
-<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org" xml:lang="en" xml:id="nanmin">
+* Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+* Copyright (C) 2000 - INRIA - Carlos Klimann
+*
+* Copyright (C) 2012 - 2016 - Scilab Enterprises
+*
+* This file is hereby licensed under the terms of the GNU GPL v2.0,
+* pursuant to article 5.3.4 of the CeCILL v.2.1.
+* This file was originally licensed under the terms of the CeCILL v2.1,
+* and continues to be available under such terms.
+* For more information, see the COPYING file which you should have received
+* along with this program.
+*
+-->
+<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink"
+          xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML"
+          xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org"
+          xml:lang="en" xml:id="nanmin">
     <refnamediv>
-        <refname>nanmin</refname>
-        <refpurpose>min  (ignoring Nan's) </refpurpose>
+        <refname>nanmin (Obsolete)</refname>
+        <refpurpose>min  (ignoring Nan's) <emphasis role="bold">This function is obsolete</emphasis>,
+          replaced with <link linkend="min">min</link>
+        </refpurpose>
     </refnamediv>
     <refsynopsisdiv>
         <title>Syntax</title>
     </refsection>
     <refsection>
         <title>Description</title>
+        <warning>
+          This function is obsolete and will  be removed from Scilab 6.1. Please use
+          <code>min()</code> instead.
+        </warning>
         <para>
             This function gives for a real or a numerical matrix <literal>x</literal>
             his largest element <literal>   m</literal> (but ignoring the
     <refsection>
         <title>Examples</title>
         <programlisting role="example"><![CDATA[
-x=[0.2113249 %nan 0.6653811;0.7560439 0.3303271 0.6283918]
-m=nanmin(x)
-m=nanmin(x,'r')
-m=nanmin(x,'c')
- ]]></programlisting>
+            x=[0.2113249 %nan 0.6653811;0.7560439 0.3303271 0.6283918]
+            m=nanmin(x)
+            m=nanmin(x,'r')
+            m=nanmin(x,'c')
+            ]]></programlisting>
     </refsection>
     <refsection>
         <title>Bibliography</title>
@@ -88,4 +97,16 @@ m=nanmin(x,'c')
             Wonacott, T.H. &amp; Wonacott, R.J.; Introductory Statistics, fifth edition, J.Wiley &amp; Sons, 1990.
         </para>
     </refsection>
+    <refsection>
+        <title>History</title>
+        <revhistory>
+            <revision>
+                <revnumber>6.0.1</revnumber>
+                <revremark>
+                    Function tagged as obsolete, to be removed from Scilab 6.1, replaced with
+                    <link linkend="min">min</link>.
+                </revremark>
+            </revision>
+        </revhistory>
+    </refsection>
 </refentry>
index 1b04388..8996e2d 100644 (file)
  * along with this program.
  *
  -->
-<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org" xml:lang="ja" xml:id="nanmax">
+<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink"
+          xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML"
+          xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org"
+          xml:lang="ja" xml:id="nanmax">
     <refnamediv>
         <refname>nanmax</refname>
-        <refpurpose>最大値  (Nanを無視) </refpurpose>
+        <refpurpose>最大値  (Nanを無視) <emphasis role="bold">This function is obsolete</emphasis>,
+          replaced with <link linkend="max">max</link>
+        </refpurpose>
     </refnamediv>
     <refsynopsisdiv>
         <title>呼び出し手順</title>
     </refsection>
     <refsection>
         <title>説明</title>
+        <warning>
+          This function is obsolete and will  be removed from Scilab 6.1. Please use
+          <code>max()</code> instead.
+        </warning>
         <para>
             この関数は,ある実数または数値行列<literal>x</literal>について,
             最大の要素<literal>m</literal> (NaNは無視)を出力します.
@@ -87,4 +96,16 @@ m=nanmax(x,'c')
             Wonacott, T.H. &amp; Wonacott, R.J.; Introductory Statistics, fifth edition, J.Wiley &amp; Sons, 1990.
         </para>
     </refsection>
+    <refsection>
+        <title>History</title>
+        <revhistory>
+            <revision>
+                <revnumber>6.0.1</revnumber>
+                <revremark>
+                  Function tagged as obsolete, to be removed from Scilab 6.1, replaced with
+                  <link linkend="max">max</link>.
+                </revremark>
+            </revision>
+        </revhistory>
+    </refsection>
 </refentry>
index 351f1b9..adf8faa 100644 (file)
  * along with this program.
  *
  -->
-<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML" xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org" xml:lang="ja" xml:id="nanmin">
+<refentry xmlns="http://docbook.org/ns/docbook" xmlns:xlink="http://www.w3.org/1999/xlink"
+          xmlns:svg="http://www.w3.org/2000/svg" xmlns:mml="http://www.w3.org/1998/Math/MathML"
+          xmlns:db="http://docbook.org/ns/docbook" xmlns:scilab="http://www.scilab.org"
+          xml:lang="ja" xml:id="nanmin">
     <refnamediv>
         <refname>nanmin</refname>
-        <refpurpose>最小値  (Nanを無視) </refpurpose>
+        <refpurpose>最小値  (Nanを無視)  <emphasis role="bold">This function is obsolete</emphasis>,
+          replaced with <link linkend="min">min</link>
+        </refpurpose>
     </refnamediv>
     <refsynopsisdiv>
         <title>呼び出し手順</title>
     </refsection>
     <refsection>
         <title>説明</title>
+        <warning>
+          This function is obsolete and will  be removed from Scilab 6.1. Please use
+          <code>min()</code> instead.
+        </warning>
         <para>
             この関数は,ある実数または数値行列<literal>x</literal>について,
             最小の要素<literal>m</literal> (NaNは無視)を出力します.
@@ -91,4 +100,16 @@ m=nanmin(x,'c')
             Wonacott, T.H. &amp; Wonacott, R.J.; Introductory Statistics, fifth edition, J.Wiley &amp; Sons, 1990.
         </para>
     </refsection>
+    <refsection>
+        <title>History</title>
+        <revhistory>
+            <revision>
+                <revnumber>6.0.1</revnumber>
+                <revremark>
+                    Function tagged as obsolete, to be removed from Scilab 6.1, replaced with
+                    <link linkend="min">min</link>.
+                </revremark>
+            </revision>
+        </revhistory>
+    </refsection>
 </refentry>
index 9a816fb..94c7bff 100644 (file)
@@ -39,6 +39,8 @@ function [s,index] = nanmax(x,orient)
     //
     // correction of bug 1239 (Bruno)
 
+    warnobsolete("max", "6.1");
+
     [lhs, rhs] = argn(0)
     if rhs == 0 then
         error(msprintf(gettext("%s: Wrong number of input arguments: %d to %d expected.\n"),"nanmax",1,2))
index bea2504..ecb65d2 100644 (file)
@@ -38,6 +38,8 @@ function [s,index] = nanmin(x,orient)
     //
     //
 
+    warnobsolete("min", "6.1");
+
     [lhs, rhs] = argn(0)
     if rhs == 0 then
         error(msprintf(gettext("%s: Wrong number of input arguments: %d to %d expected.\n"),"nanmin",1,2))
diff --git a/scilab/modules/statistics/tests/nonreg_tests/bug_1239.dia.ref b/scilab/modules/statistics/tests/nonreg_tests/bug_1239.dia.ref
deleted file mode 100644 (file)
index c146228..0000000
+++ /dev/null
@@ -1,33 +0,0 @@
-//<-- CLI SHELL MODE -->
-// =============================================================================
-// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
-// Copyright (C) 2005-2008 - INRIA -Pierre MARECHAL <pierre.marechal@inria.fr>
-//
-//  This file is distributed under the same license as the Scilab package.
-// =============================================================================
-// <-- Non-regression test for bug 1239 -->
-//
-// <-- Bugzilla URL -->
-// http://bugzilla.scilab.org/show_bug.cgi?id=1239
-//
-// <-- Short Description -->
-//    This bug was found by Marek. He says us in the newsgroup:
-//
-//    It seems that nanmax, nanmin functions are not working well,
-//    or at
-//    least not according to documentation(they should supposedly
-//    ignore %nan
-//    values). I get the wollowing results in scilab 3.0:
-//
-//    -->nanmax([%nan, -1])
-//     ans  = Inf
-//
-//    -->nanmax([%nan, 1])
-//     ans  = Inf
-//    The same for nanmin. 
-//    ========================================================
-//
-//    Btw, on a scilab cvs I don't see any problem with nanmin.
-//
-//     Bruno
-if( (nanmax([%nan, -1]) <> -1) | (nanmax([%nan, 1]) <> 1) | (nanmin([%nan, -1]) <> -1) | (nanmin([%nan, 1]) <> 1) ) then bugmes();quit;end
index 07bc6e4..8b4066c 100644 (file)
@@ -6,6 +6,9 @@
 //  This file is distributed under the same license as the Scilab package.
 // =============================================================================
 
+// <-- CLI SHELL MODE -->
+// <-- NO CHECK REF -->
+
 // <-- Non-regression test for bug 1239 -->
 //
 // <-- Bugzilla URL -->
@@ -25,7 +28,7 @@
 //
 //    -->nanmax([%nan, 1])
 //     ans  = Inf
-//    The same for nanmin. 
+//    The same for nanmin.
 //    ========================================================
 //
 //    Btw, on a scilab cvs I don't see any problem with nanmin.