Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates
 72.14.220.158 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=Dgv2h5GX/oQrsq2KfE98Enp50kqQU/jR5oRCa0rZ3YVULR43ybYhn4PUVnMHj4xkcFLgJwrp0ex5NK9aDIuf0W+Ym7mX/3JxgHwyXnvvkCNLLaBaXqAXtYluaMguQX+RY0cMBRmXH4lNcSJQJTek17h9TqxXndYYdtwI+AFQJ8E=
Message-ID: <25aac9fc0804240645t3c34c253p4e706e9564396a41@mail.gmail.com>
Date: Thu, 24 Apr 2008 14:45:03 +0100
From: sebb <sebbaz@gmail.com>
To: "Commons Developers List" <dev@commons.apache.org>
Subject: Re: svn commit: r651244 - in
 /commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear:
 BigMatrixImpl.java RealMatrixImpl.java
In-Reply-To: <1209041260.4810816c44fe3@imp.free.fr>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <20080424122945.C3D6A1A9832@eris.apache.org>
	 <1209041260.4810816c44fe3@imp.free.fr>

2008/4/24  <luc.maisonobe@free.fr>:
> This change is a little weird.
>
>  We discussed about this field back in january
>  (http://markmail.org/message/45y6znvvg2ucauwa) and decided to deprecate it
>  before changing it to private static.
>
>  We forgot to mark it deprecated while releasing 1.2.
>
>  Since 2.0 will introduce other incompatible changes, I thought it was time to
>  make the change and went ahead with the modification. I feel uncomfortable with
>  this though, because users have not been warned about this (however, I'm not
>  sure anybody relied on the availability of this protected field).
>
>  There is another change that bother me: we did not deprecate either the
>  UnivariateRealSolverFactory class and did not replace it by setter injection in
>  1.2, as we have done for other algorithms.
>
>  Do you think we should:
>   1) revert the change and go back to a protected field, not
>     forgetting to mark it as deprecated and removing it in 3.0
>   2) go ahead and only apologize about forgotten deprecation mark
>     (and also replace UnivariateRealSolverFactory in the same move)
>   3) do a 1.3 release only in order to add the forgotten deprecation
>
>  Choice 1 is the most conservative one, but means we will keep dirty code for a
>  long time. Choice 2 is my personal preference, it is rather rude but I think we
>  made a mistake and have to assume it. Choice 3 seems realistic only if we
>  release 1.3 very soon to let users take these deprecations onto account and wait
>  some time before 2.0, I don't like it because 2.0 is really badly needed for the
>  features it will bring.

Upgrading from 1.2 to 2.0 is a major version change - seems to me that
users should expect such changes.
So long as the change is clearly documented, I don't see a problem, so
I would support choice 2.

>  Luc
>
>  Selon luc@apache.org:
>
>
>
>  > Author: luc
>  > Date: Thu Apr 24 05:29:41 2008
>  > New Revision: 651244
>  >
>  > URL: http://svn.apache.org/viewvc?rev=651244&view=rev
>  > Log:
>  > changed the field TOO_SMALL from protected to private static
>  >
>  > Modified:
>  >
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/BigMatrixImpl.java
>  >
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/RealMatrixImpl.java
>  >
>  > Modified:
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/BigMatrixImpl.java
>  > URL:
>  >
>  http://svn.apache.org/viewvc/commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/BigMatrixImpl.java?rev=651244&r1=651243&r2=651244&view=diff
>  > ==============================================================================
>  > ---
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/BigMatrixImpl.java
>  > (original)
>  > +++
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/BigMatrixImpl.java
>  > Thu Apr 24 05:29:41 2008
>  > @@ -73,7 +73,7 @@
>  >      private int scale = 64;
>  >
>  >      /** Bound to determine effective singularity in LU decomposition */
>  > -    protected static BigDecimal TOO_SMALL = new BigDecimal(10E-12);
>  > +    private static final BigDecimal TOO_SMALL = new BigDecimal(10E-12);
>  >
>  >      /** BigDecimal 0 */
>  >      static final BigDecimal ZERO = new BigDecimal(0);
>  >
>  > Modified:
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/RealMatrixImpl.java
>  > URL:
>  >
>  http://svn.apache.org/viewvc/commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/RealMatrixImpl.java?rev=651244&r1=651243&r2=651244&view=diff
>  > ==============================================================================
>  > ---
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/RealMatrixImpl.java
>  > (original)
>  > +++
>  >
>  commons/proper/math/branches/MATH_2_0/src/java/org/apache/commons/math/linear/RealMatrixImpl.java
>  > Thu Apr 24 05:29:41 2008
>  > @@ -69,7 +69,7 @@
>  >      private int parity = 1;
>  >
>  >      /** Bound to determine effective singularity in LU decomposition */
>  > -    protected static double TOO_SMALL = 10E-12;
>  > +    private static final double TOO_SMALL = 10E-12;
>  >
>  >      /**
>  >       * Creates a matrix with no data
>  >
>  >
>  >
>
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>  For additional commands, e-mail: dev-help@commons.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org