Author: erans
Date: Mon Dec  6 11:56:26 2010
New Revision: 1042596

URL: http://svn.apache.org/viewvc?rev=1042596&view=rev
Log:
MATH-452
Made all static variables (accuracies) "private".

Modified:
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseAbstractUnivariateRealSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BisectionSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BrentSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/LaguerreSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver2.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/NewtonSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/RiddersSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/SecantSolver.java
    
commons/proper/math/trunk/src/main/java/org/apache/commons/math/distribution/AbstractContinuousDistribution.java

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseAbstractUnivariateRealSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseAbstractUnivariateRealSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseAbstractUnivariateRealSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseAbstractUnivariateRealSolver.java
 Mon Dec  6 11:56:26 2010
@@ -34,12 +34,10 @@ import org.apache.commons.math.analysis.
  */
 public abstract class BaseAbstractUnivariateRealSolver<FUNC extends 
UnivariateRealFunction>
     implements BaseUnivariateRealSolver<FUNC> {
-    /** Default absolute accuracy */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
     /** Default relative accuracy. */
-    public static final double DEFAULT_RELATIVE_ACCURACY = 1e-14;
+    private static final double DEFAULT_RELATIVE_ACCURACY = 1e-14;
     /** Default function value accuracy. */
-    public static final double DEFAULT_FUNCTION_VALUE_ACCURACY = 1e-15;
+    private static final double DEFAULT_FUNCTION_VALUE_ACCURACY = 1e-15;
     /** Function value accuracy. */
     private final double functionValueAccuracy;
     /** Absolute accuracy. */

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BisectionSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BisectionSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BisectionSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BisectionSolver.java
 Mon Dec  6 11:56:26 2010
@@ -28,10 +28,10 @@ import org.apache.commons.math.util.Fast
  */
 public class BisectionSolver extends AbstractUnivariateRealSolver {
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracy.
+     * Construct a solver with default accuracy (1e-6).
      */
     public BisectionSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BrentSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BrentSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BrentSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BrentSolver.java
 Mon Dec  6 11:56:26 2010
@@ -37,10 +37,10 @@ public class BrentSolver extends Abstrac
     /** Serializable version identifier */
     private static final long serialVersionUID = 7694577816772532779L;
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracies.
+     * Construct a solver with default accuracy (1e-6).
      */
     public BrentSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/LaguerreSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/LaguerreSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/LaguerreSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/LaguerreSolver.java
 Mon Dec  6 11:56:26 2010
@@ -40,12 +40,12 @@ import org.apache.commons.math.util.Fast
  */
 public class LaguerreSolver extends AbstractPolynomialSolver {
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
     /** Complex solver. */
     protected ComplexSolver complexSolver = new ComplexSolver();
 
     /**
-     * Construct a solver with default accuracies.
+     * Construct a solver with default accuracy (1e-6).
      */
     public LaguerreSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver.java
 Mon Dec  6 11:56:26 2010
@@ -49,10 +49,10 @@ public class MullerSolver extends Abstra
     /** Serializable version identifier */
     private static final long serialVersionUID = 7694577816772532779L;
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracies.
+     * Construct a solver with default accuracy (1e-6).
      */
     public MullerSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver2.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver2.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver2.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/MullerSolver2.java
 Mon Dec  6 11:56:26 2010
@@ -49,10 +49,10 @@ public class MullerSolver2 extends Abstr
     /** Serializable version identifier */
     private static final long serialVersionUID = 7694577816772532779L;
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracies.
+     * Construct a solver with default accuracy (1e-6).
      */
     public MullerSolver2() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/NewtonSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/NewtonSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/NewtonSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/NewtonSolver.java
 Mon Dec  6 11:56:26 2010
@@ -30,7 +30,7 @@ import org.apache.commons.math.util.Fast
  */
 public class NewtonSolver extends AbstractDifferentiableUnivariateRealSolver {
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
      * Construct a solver.

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/RiddersSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/RiddersSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/RiddersSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/RiddersSolver.java
 Mon Dec  6 11:56:26 2010
@@ -33,10 +33,10 @@ import org.apache.commons.math.util.Math
  */
 public class RiddersSolver extends AbstractUnivariateRealSolver {
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracy.
+     * Construct a solver with default accuracy (1e-6).
      */
     public RiddersSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/SecantSolver.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/SecantSolver.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/SecantSolver.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/SecantSolver.java
 Mon Dec  6 11:56:26 2010
@@ -38,10 +38,10 @@ import org.apache.commons.math.util.Fast
  */
 public class SecantSolver extends AbstractUnivariateRealSolver {
     /** Default absolute accuracy. */
-    public static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
+    private static final double DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
 
     /**
-     * Construct a solver with default accuracy.
+     * Construct a solver with default accuracy (1e-6).
      */
     public SecantSolver() {
         this(DEFAULT_ABSOLUTE_ACCURACY);

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/distribution/AbstractContinuousDistribution.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/distribution/AbstractContinuousDistribution.java?rev=1042596&r1=1042595&r2=1042596&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/distribution/AbstractContinuousDistribution.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/distribution/AbstractContinuousDistribution.java
 Mon Dec  6 11:56:26 2010
@@ -42,16 +42,18 @@ public abstract class AbstractContinuous
     implements ContinuousDistribution, Serializable {
     /** Serializable version identifier */
     private static final long serialVersionUID = -38038050983108802L;
+    /** Default accuracy. */
+    public static final double SOLVER_DEFAULT_ABSOLUTE_ACCURACY = 1e-6;
     /**
      * RandomData instance used to generate samples from the distribution
      * @since 2.2
      */
     protected final RandomDataImpl randomData = new RandomDataImpl();
     /**
-     * Solver absolute accuracy for inverse cumulative computation
+     * Solver absolute accuracy for inverse cumulative computation.
      * @since 2.1
      */
-    private double solverAbsoluteAccuracy = 
BrentSolver.DEFAULT_ABSOLUTE_ACCURACY;
+    private double solverAbsoluteAccuracy = SOLVER_DEFAULT_ABSOLUTE_ACCURACY;
     /**
      * Default constructor.
      */
@@ -125,7 +127,7 @@ public abstract class AbstractContinuous
         // find root
         double root = UnivariateRealSolverUtils.solve(rootFindingFunction,
                 // override getSolverAbsoluteAccuracy() to use a Brent solver 
with
-                // absolute accuracy different from BrentSolver default
+                // absolute accuracy different from the default.
                 bracket[0],bracket[1], getSolverAbsoluteAccuracy());
         return root;
     }
@@ -209,6 +211,8 @@ public abstract class AbstractContinuous
 
     /**
      * Returns the solver absolute accuracy for inverse cumulative computation.
+     * You can override this method in order to use a Brent solver with an
+     * absolute accuracy different from the default.
      *
      * @return the maximum absolute error in inverse cumulative probability 
estimates
      * @since 2.1
@@ -216,5 +220,4 @@ public abstract class AbstractContinuous
     protected double getSolverAbsoluteAccuracy() {
         return solverAbsoluteAccuracy;
     }
-
 }


Reply via email to