Repository: commons-math
Updated Branches:
  refs/heads/master a51908af1 -> b5e155e7d


http://git-wip-us.apache.org/repos/asf/commons-math/blob/d8bfc8c8/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunctionTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunctionTest.java
 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunctionTest.java
index 8c78aed..773a948 100644
--- 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunctionTest.java
+++ 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunctionTest.java
@@ -16,435 +16,147 @@
  */
 package org.apache.commons.math3.analysis.interpolation;
 
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+
 import org.apache.commons.math3.exception.DimensionMismatchException;
-import org.apache.commons.math3.exception.MathIllegalArgumentException;
-import org.apache.commons.math3.exception.OutOfRangeException;
+import org.apache.commons.math3.exception.InsufficientDataException;
+import org.apache.commons.math3.exception.NonMonotonicSequenceException;
+import org.apache.commons.math3.exception.NullArgumentException;
 import org.apache.commons.math3.analysis.BivariateFunction;
 import org.apache.commons.math3.distribution.UniformRealDistribution;
 import org.apache.commons.math3.random.RandomGenerator;
 import org.apache.commons.math3.random.Well19937c;
+import org.apache.commons.math3.util.FastMath;
+import org.apache.commons.math3.util.Precision;
 import org.junit.Assert;
 import org.junit.Test;
-import org.junit.Ignore;
 
 /**
  * Test case for the bicubic function.
- * 
  */
-public final class BicubicSplineInterpolatingFunctionTest {
+public final class BicubicSplineInterpolatingFunctionTest
+{
     /**
      * Test preconditions.
      */
     @Test
-    public void testPreconditions() {
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2.5};
+    public void testPreconditions()
+    {
+        double[] xval = new double[] { 3, 4, 5, 6.5, 7.5 };
+        double[] yval = new double[] { -4, -3, -1, 2.5, 3.5 };
         double[][] zval = new double[xval.length][yval.length];
 
         @SuppressWarnings("unused")
-        BivariateFunction bcf = new BicubicSplineInterpolatingFunction(xval, 
yval, zval,
-                                                                           
zval, zval, zval);
+        BicubicSplineInterpolatingFunction bcf = new 
BicubicSplineInterpolatingFunction( xval, yval, zval );
         
-        double[] wxval = new double[] {3, 2, 5, 6.5};
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(wxval, yval, zval, 
zval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (MathIllegalArgumentException e) {
-            // Expected
-        }
-        double[] wyval = new double[] {-4, -1, -1, 2.5};
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, wyval, zval, 
zval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (MathIllegalArgumentException e) {
-            // Expected
-        }
-        double[][] wzval = new double[xval.length][yval.length - 1];
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, wzval, 
zval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
wzval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
zval, wzval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
zval, zval, wzval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-
-        wzval = new double[xval.length - 1][yval.length];
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, wzval, 
zval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
wzval, zval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
zval, wzval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        try {
-            bcf = new BicubicSplineInterpolatingFunction(xval, yval, zval, 
zval, zval, wzval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
+        try
+        {
+            bcf = new BicubicSplineInterpolatingFunction( null, yval, zval );
+            Assert.fail( "Failed to detect x null pointer" );
+        }
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
     }
 
-    /**
-     * Test for a plane.
-     * <p>
-     * z = 2 x - 3 y + 5
-     */
-    @Ignore@Test
-    public void testPlane() {
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2, 2.5};
-        // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 2 * x - 3 * y + 5;
-                }
-            };
-        double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                zval[i][j] = f.value(xval[i], yval[j]);
-            }
-        }
-        // Partial derivatives with respect to x
-        double[][] dZdX = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdX[i][j] = 2;
-            }
+        try
+        {
+            bcf = new BicubicSplineInterpolatingFunction( xval, null, zval );
+            Assert.fail( "Failed to detect y null pointer" );
         }
-        // Partial derivatives with respect to y
-        double[][] dZdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdY[i][j] = -3;
-            }
-        }
-        // Partial cross-derivatives
-        double[][] dZdXdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdXdY[i][j] = 0;
-            }
-        }
-
-        BivariateFunction bcf = new BicubicSplineInterpolatingFunction(xval, 
yval, zval,
-                                                                           
dZdX, dZdY, dZdXdY);
-        double x, y;
-        double expected, result;
-
-        x = 4;
-        y = -3;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("On sample point",
-                            expected, result, 1e-15);
-
-        x = 4.5;
-        y = -1.5;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("Half-way between sample points (middle of the 
patch)",
-                            expected, result, 0.3);
-
-        x = 3.5;
-        y = -3.5;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("Half-way between sample points (border of the 
patch)",
-                            expected, result, 0.3);
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
     }
 
-    /**
-     * Test for a paraboloid.
-     * <p>
-     * z = 2 x<sup>2</sup> - 3 y<sup>2</sup> + 4 x y - 5
-     */
-    @Ignore@Test
-    public void testParaboloid() {
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2, 2.5};
-        // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 2 * x * x - 3 * y * y + 4 * x * y - 5;
-                }
-            };
-        double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                zval[i][j] = f.value(xval[i], yval[j]);
-            }
-        }
-        // Partial derivatives with respect to x
-        double[][] dZdX = new double[xval.length][yval.length];
-        BivariateFunction dfdX = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 4 * (x + y);
-                }
-            };
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdX[i][j] = dfdX.value(xval[i], yval[j]);
-            }
+        try
+        {
+            bcf = new BicubicSplineInterpolatingFunction( xval, yval, null );
+            Assert.fail( "Failed to detect z null pointer" );
         }
-        // Partial derivatives with respect to y
-        double[][] dZdY = new double[xval.length][yval.length];
-        BivariateFunction dfdY = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 4 * x - 6 * y;
-                }
-            };
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdY[i][j] = dfdY.value(xval[i], yval[j]);
-            }
-        }
-        // Partial cross-derivatives
-        double[][] dZdXdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdXdY[i][j] = 4;
-            }
-        }
-
-        BivariateFunction bcf = new BicubicSplineInterpolatingFunction(xval, 
yval, zval,
-                                                                           
dZdX, dZdY, dZdXdY);
-        double x, y;
-        double expected, result;
-        
-        x = 4;
-        y = -3;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("On sample point",
-                            expected, result, 1e-15);
-
-        x = 4.5;
-        y = -1.5;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("Half-way between sample points (middle of the 
patch)",
-                            expected, result, 2);
-
-        x = 3.5;
-        y = -3.5;
-        expected = f.value(x, y);
-        result = bcf.value(x, y);
-        Assert.assertEquals("Half-way between sample points (border of the 
patch)",
-                            expected, result, 2);
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
     }
 
-    /**
-     * Test for partial derivatives of {@link BicubicSplineFunction}.
-     * <p>
-     * f(x, y) = &Sigma;<sub>i</sub>&Sigma;<sub>j</sub> (i+1) (j+2) 
x<sup>i</sup> y<sup>j</sup>
-     */
-    @Ignore@Test
-    public void testSplinePartialDerivatives() {
-        final int N = 4;
-        final double[] coeff = new double[16];
-
-        for (int i = 0; i < N; i++) {
-            for (int j = 0; j < N; j++) {
-                coeff[i + N * j] = (i + 1) * (j + 2);
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 2.0, 3.0 };
+            bcf = new BicubicSplineInterpolatingFunction( xval1, yval, zval );
+            Assert.fail( "Failed to detect insufficient x data" );
             }
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
         }
 
-        final BicubicSplineFunction f = new BicubicSplineFunction(coeff);
-        BivariateFunction derivative;
-        final double x = 0.435;
-        final double y = 0.776;
-        final double tol = 1e-13;
-
-        derivative = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double y2 = y * y;
-                    final double y3 = y2 * y;
-                    final double yFactor = 2 + 3 * y + 4 * y2 + 5 * y3;
-                    return yFactor * (2 + 6 * x + 12 * x2);
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 2.0, 3.0 };
+            bcf = new BicubicSplineInterpolatingFunction( xval, yval1, zval );
+            Assert.fail( "Failed to detect insufficient y data" );
                 }
-            };
-        Assert.assertEquals("dFdX", derivative.value(x, y),
-                            f.partialDerivativeX().value(x, y), tol);
-        
-        derivative = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double x3 = x2 * x;
-                    final double y2 = y * y;
-                    final double xFactor = 1 + 2 * x + 3 * x2 + 4 * x3;
-                    return xFactor * (3 + 8 * y + 15 * y2);
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
                 }
-            };
-        Assert.assertEquals("dFdY", derivative.value(x, y),
-                            f.partialDerivativeY().value(x, y), tol);
 
-        derivative = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double y2 = y * y;
-                    final double y3 = y2 * y;
-                    final double yFactor = 2 + 3 * y + 4 * y2 + 5 * y3;
-                    return yFactor * (6 + 24 * x);
+        try
+        {
+            double zval1[][] = new double[4][4];
+            bcf = new BicubicSplineInterpolatingFunction( xval, yval, zval1 );
+            Assert.fail( "Failed to detect insufficient z data" );
                 }
-            };
-        Assert.assertEquals("d2FdX2", derivative.value(x, y),
-                            f.partialDerivativeXX().value(x, y), tol);
-
-        derivative = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double x3 = x2 * x;
-                    final double xFactor = 1 + 2 * x + 3 * x2 + 4 * x3;
-                    return xFactor * (8 + 30 * y);
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
                 }
-            };
-        Assert.assertEquals("d2FdY2", derivative.value(x, y),
-                            f.partialDerivativeYY().value(x, y), tol);
 
-        derivative = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double y2 = y * y;
-                    final double yFactor = 3 + 8 * y + 15 * y2;
-                    return yFactor * (2 + 6 * x + 12 * x2);
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0 };
+            bcf = new BicubicSplineInterpolatingFunction( xval1, yval, zval );
+            Assert.fail( "Failed to detect data set array with different 
sizes." );
                 }
-            };
-        Assert.assertEquals("d2FdXdY", derivative.value(x, y),
-                            f.partialDerivativeXY().value(x, y), tol);
+        catch ( DimensionMismatchException iae )
+        {
+            // Expected.
     }
 
-    /**
-     * Test that the partial derivatives computed from a
-     * {@link BicubicSplineInterpolatingFunction} match the input data.
-     * <p>
-     * f(x, y) = 5
-     *           - 3 x + 2 y
-     *           - x y + 2 x<sup>2</sup> - 3 y<sup>2</sup>
-     *           + 4 x<sup>2</sup> y - x y<sup>2</sup> - 3 x<sup>3</sup> + 
y<sup>3</sup>
-     */
-    @Ignore@Test
-    public void testMatchingPartialDerivatives() {
-        final int sz = 21;
-        double[] val = new double[sz];
-        // Coordinate values
-        final double delta = 1d / (sz - 1);
-        for (int i = 0; i < sz; i++) {
-            val[i] = i * delta;
-        }
-        // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double x3 = x2 * x;
-                    final double y2 = y * y;
-                    final double y3 = y2 * y;
-
-                    return 5
-                        - 3 * x + 2 * y
-                        - x * y + 2 * x2 - 3 * y2
-                        + 4 * x2 * y - x * y2 - 3 * x3 + y3;
-                }
-            };
-        double[][] fval = new double[sz][sz];
-        for (int i = 0; i < sz; i++) {
-            for (int j = 0; j < sz; j++) {
-                fval[i][j] = f.value(val[i], val[j]);
-            }
-        }
-        // Partial derivatives with respect to x
-        double[][] dFdX = new double[sz][sz];
-        BivariateFunction dfdX = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double y2 = y * y;                    
-                    return - 3 - y + 4 * x + 8 * x * y - y2 - 9 * x2;
-                }
-            };
-        for (int i = 0; i < sz; i++) {
-            for (int j = 0; j < sz; j++) {
-                dFdX[i][j] = dfdX.value(val[i], val[j]);
-            }
-        }
-        // Partial derivatives with respect to y
-        double[][] dFdY = new double[sz][sz];
-        BivariateFunction dfdY = new BivariateFunction() {
-                public double value(double x, double y) {
-                    final double x2 = x * x;
-                    final double y2 = y * y;                    
-                    return 2 - x - 6 * y + 4 * x2 - 2 * x * y + 3 * y2;
-                }
-            };
-        for (int i = 0; i < sz; i++) {
-            for (int j = 0; j < sz; j++) {
-                dFdY[i][j] = dfdY.value(val[i], val[j]);
-            }
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0 };
+            bcf = new BicubicSplineInterpolatingFunction( xval, yval1, zval );
+            Assert.fail( "Failed to detect data set array with different 
sizes." );
         }
-        // Partial cross-derivatives
-        double[][] d2FdXdY = new double[sz][sz];
-        BivariateFunction d2fdXdY = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return -1 + 8 * x - 2 * y;
+        catch ( DimensionMismatchException iae )
+        {
+            // Expected.
                 }
-            };
-        for (int i = 0; i < sz; i++) {
-            for (int j = 0; j < sz; j++) {
-                d2FdXdY[i][j] = d2fdXdY.value(val[i], val[j]);
+
+        // X values not sorted.
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 0.5, 7.0, 3.5 };
+            bcf = new BicubicSplineInterpolatingFunction( xval1, yval, zval );
+            Assert.fail( "Failed to detect unsorted x arguments." );
             }
+        catch ( NonMonotonicSequenceException iae )
+        {
+            // Expected.
         }
 
-        BicubicSplineInterpolatingFunction bcf
-            = new BicubicSplineInterpolatingFunction(val, val, fval, dFdX, 
dFdY, d2FdXdY);
-
-        double x, y;
-        double expected, result;
-
-        final double tol = 1e-12;
-        for (int i = 0; i < sz; i++) {
-            x = val[i];
-            for (int j = 0; j < sz; j++) {
-                y = val[j];
-                
-                expected = dfdX.value(x, y);
-                result = bcf.partialDerivativeX(x, y);
-                Assert.assertEquals(x + " " + y + " dFdX", expected, result, 
tol);
-
-                expected = dfdY.value(x, y);
-                result = bcf.partialDerivativeY(x, y);
-                Assert.assertEquals(x + " " + y + " dFdY", expected, result, 
tol);
-                
-                expected = d2fdXdY.value(x, y);
-                result = bcf.partialDerivativeXY(x, y);
-                Assert.assertEquals(x + " " + y + " d2FdXdY", expected, 
result, tol);
+        // Y values not sorted.
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 1.5, 0.0, 3.0 };
+            bcf = new BicubicSplineInterpolatingFunction( xval, yval1, zval );
+            Assert.fail( "Failed to detect unsorted y arguments." );
             }
+        catch ( NonMonotonicSequenceException iae )
+        {
+            // Expected.
         }
     }
 
@@ -454,73 +166,28 @@ public final class BicubicSplineInterpolatingFunctionTest 
{
      * z = 2 x - 3 y + 5
      */
     @Test
-    public void testInterpolation1() {
-        final int sz = 21;
-        double[] xval = new double[sz];
-        double[] yval = new double[sz];
-        // Coordinate values
-        final double delta = 1d / (sz - 1);
-        for (int i = 0; i < sz; i++) {
-            xval[i] = -1 + 15 * i * delta;
-            yval[i] = -20 + 30 * i * delta;
-        }
+    public void testInterpolatePlane()
+    {
+        final int numberOfElements = 10;
+        final double minimumX = -10;
+        final double maximumX = 10;
+        final double minimumY = -10;
+        final double maximumY = 10;
+        final int numberOfSamples = 100;
+        final double interpolationTolerance = 7e-15;
+        final double maxTolerance = 6e-14;
 
         // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
+        BivariateFunction f = new BivariateFunction()
+        {
+            public double value( double x, double y )
+            {
                     return 2 * x - 3 * y + 5;
                 }
             };
-        double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                zval[i][j] = f.value(xval[i], yval[j]);
-            }
-        }
-        // Partial derivatives with respect to x
-        double[][] dZdX = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdX[i][j] = 2;
-            }
-        }
-        // Partial derivatives with respect to y
-        double[][] dZdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdY[i][j] = -3;
-            }
-        }
-        // Partial cross-derivatives
-        double[][] dZdXdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdXdY[i][j] = 0;
-            }
-        }
-
-        final BivariateFunction bcf
-            = new BicubicSplineInterpolatingFunction(xval, yval, zval,
-                                                     dZdX, dZdY, dZdXdY);
-        double x, y;
 
-        final RandomGenerator rng = new Well19937c(1234567L); // "tol" depends 
on the seed.
-        final UniformRealDistribution distX
-            = new UniformRealDistribution(rng, xval[0], xval[xval.length - 1]);
-        final UniformRealDistribution distY
-            = new UniformRealDistribution(rng, yval[0], yval[yval.length - 1]);
-
-        final int numSamples = 50;
-        final double tol = 6;
-        for (int i = 0; i < numSamples; i++) {
-            x = distX.sample();
-            for (int j = 0; j < numSamples; j++) {
-                y = distY.sample();
-//                 System.out.println(x + " " + y + " " + f.value(x, y) + " " 
+ bcf.value(x, y));
-                Assert.assertEquals(f.value(x, y),  bcf.value(x, y), tol);
-            }
-//             System.out.println();
-        }
+        testInterpolation( minimumX, maximumX, minimumY, maximumY, 
numberOfElements, numberOfSamples, f,
+                           interpolationTolerance, maxTolerance );
     }
 
     /**
@@ -529,140 +196,85 @@ public final class 
BicubicSplineInterpolatingFunctionTest {
      * z = 2 x<sup>2</sup> - 3 y<sup>2</sup> + 4 x y - 5
      */
     @Test
-    public void testInterpolation2() {
-        final int sz = 21;
-        double[] xval = new double[sz];
-        double[] yval = new double[sz];
-        // Coordinate values
-        final double delta = 1d / (sz - 1);
-        for (int i = 0; i < sz; i++) {
-            xval[i] = -1 + 15 * i * delta;
-            yval[i] = -20 + 30 * i * delta;
-        }
+    public void testInterpolationParabaloid()
+    {
+        final int numberOfElements = 10;
+        final double minimumX = -10;
+        final double maximumX = 10;
+        final double minimumY = -10;
+        final double maximumY = 10;
+        final int numberOfSamples = 100;
+        final double interpolationTolerance = 2e-14;
+        final double maxTolerance = 6e-14;
 
         // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
+        BivariateFunction f = new BivariateFunction()
+        {
+            public double value( double x, double y )
+            {
                     return 2 * x * x - 3 * y * y + 4 * x * y - 5;
                 }
             };
-        double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                zval[i][j] = f.value(xval[i], yval[j]);
-            }
-        }
-        // Partial derivatives with respect to x
-        double[][] dZdX = new double[xval.length][yval.length];
-        BivariateFunction dfdX = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 4 * (x + y);
-                }
-            };
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdX[i][j] = dfdX.value(xval[i], yval[j]);
-            }
-        }
-        // Partial derivatives with respect to y
-        double[][] dZdY = new double[xval.length][yval.length];
-        BivariateFunction dfdY = new BivariateFunction() {
-                public double value(double x, double y) {
-                    return 4 * x - 6 * y;
-                }
-            };
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdY[i][j] = dfdY.value(xval[i], yval[j]);
-            }
-        }
-        // Partial cross-derivatives
-        double[][] dZdXdY = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
-                dZdXdY[i][j] = 4;
+
+        testInterpolation( minimumX, maximumX, minimumY, maximumY, 
numberOfElements, numberOfSamples, f,
+                           interpolationTolerance, maxTolerance );
+        }
+
+    private void testInterpolation( double minimumX, double maximumX, double 
minimumY, double maximumY,
+                                    int numberOfElements, int numberOfSamples, 
BivariateFunction f, double tolerance,
+                                    double maxTolerance )
+    {
+        double expected;
+        double actual;
+        double currentX;
+        double currentY;
+        final double deltaX = ( maximumX - minimumX ) / ( (double) 
numberOfElements );
+        final double deltaY = ( maximumY - minimumY ) / ( (double) 
numberOfElements );
+        double xValues[] = new double[numberOfElements];
+        double yValues[] = new double[numberOfElements];
+        double zValues[][] = new double[numberOfElements][numberOfElements];
+
+        for ( int i = 0; i < numberOfElements; i++ )
+        {
+            xValues[i] = minimumX + deltaX * (double) i;
+            for ( int j = 0; j < numberOfElements; j++ )
+            {
+                yValues[j] = minimumY + deltaY * (double) j;
+                zValues[i][j] = f.value( xValues[i], yValues[j] );
             }
         }
 
-        BivariateFunction bcf = new BicubicSplineInterpolatingFunction(xval, 
yval, zval,
-                                                                       dZdX, 
dZdY, dZdXdY);
-        double x, y;
+        BivariateFunction interpolation = new 
BicubicSplineInterpolatingFunction( xValues, yValues, zValues );
 
-        final RandomGenerator rng = new Well19937c(1234567L); // "tol" depends 
on the seed.
-        final UniformRealDistribution distX
-            = new UniformRealDistribution(rng, xval[0], xval[xval.length - 1]);
-        final UniformRealDistribution distY
-            = new UniformRealDistribution(rng, yval[0], yval[yval.length - 1]);
-
-        final double tol = 224;
-        for (int i = 0; i < sz; i++) {
-            x = distX.sample();
-            for (int j = 0; j < sz; j++) {
-                y = distY.sample();
-//                 System.out.println(x + " " + y + " " + f.value(x, y) + " " 
+ bcf.value(x, y));
-                Assert.assertEquals(f.value(x, y),  bcf.value(x, y), tol);
+        for ( int i = 0; i < numberOfElements; i++ )
+        {
+            currentX = xValues[i];
+            for ( int j = 0; j < numberOfElements; j++ )
+            {
+                currentY = yValues[j];
+                expected = f.value( currentX, currentY );
+                actual = interpolation.value( currentX, currentY );
+                assertTrue( Precision.equals( expected, actual ) );
             }
-//             System.out.println();
         }
-    }
 
-    @Test
-    public void testIsValidPoint() {
-        final double xMin = -12;
-        final double xMax = 34;
-        final double yMin = 5;
-        final double yMax = 67;
-        final double[] xval = new double[] { xMin, xMax };
-        final double[] yval = new double[] { yMin, yMax };
-        final double[][] f = new double[][] { { 1, 2 },
-                                              { 3, 4 } };
-        final double[][] dFdX = f;
-        final double[][] dFdY = f;
-        final double[][] dFdXdY = f;
-
-        final BicubicSplineInterpolatingFunction bcf
-            = new BicubicSplineInterpolatingFunction(xval, yval, f,
-                                                     dFdX, dFdY, dFdXdY);
-
-        double x, y;
-
-        x = xMin;
-        y = yMin;
-        Assert.assertTrue(bcf.isValidPoint(x, y));
-        // Ensure that no exception is thrown.
-        bcf.value(x, y);
-
-        x = xMax;
-        y = yMax;
-        Assert.assertTrue(bcf.isValidPoint(x, y));
-        // Ensure that no exception is thrown.
-        bcf.value(x, y);
- 
-        final double xRange = xMax - xMin;
-        final double yRange = yMax - yMin;
-        x = xMin + xRange / 3.4;
-        y = yMin + yRange / 1.2;
-        Assert.assertTrue(bcf.isValidPoint(x, y));
-        // Ensure that no exception is thrown.
-        bcf.value(x, y);
-
-        final double small = 1e-8;
-        x = xMin - small;
-        y = yMax;
-        Assert.assertFalse(bcf.isValidPoint(x, y));
-        // Ensure that an exception would have been thrown.
-        try {
-            bcf.value(x, y);
-            Assert.fail("OutOfRangeException expected");
-        } catch (OutOfRangeException expected) {}
+        final RandomGenerator rng = new Well19937c( 1234567L ); // "tol" 
depends on the seed.
+        final UniformRealDistribution distX =
+            new UniformRealDistribution( rng, xValues[0], 
xValues[xValues.length - 1] );
+        final UniformRealDistribution distY =
+            new UniformRealDistribution( rng, yValues[0], 
yValues[yValues.length - 1] );
+
+        double sumError = 0;
+        for ( int i = 0; i < numberOfSamples; i++ )
+        {
+            currentX = distX.sample();
+            currentY = distY.sample();
+            expected = f.value( currentX, currentY );
+            actual = interpolation.value( currentX, currentY );
+            sumError += FastMath.abs( actual - expected );
+            assertEquals( expected, actual, maxTolerance );
+    }
 
-        x = xMin;
-        y = yMax + small;
-        Assert.assertFalse(bcf.isValidPoint(x, y));
-        // Ensure that an exception would have been thrown.
-        try {
-            bcf.value(x, y);
-            Assert.fail("OutOfRangeException expected");
-        } catch (OutOfRangeException expected) {}
+        assertEquals( 0.0, ( sumError / (double) numberOfSamples ), tolerance 
);
     }
 }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/d8bfc8c8/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatorTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatorTest.java
 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatorTest.java
index c4a56bb..69d52a1 100644
--- 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatorTest.java
+++ 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatorTest.java
@@ -17,7 +17,10 @@
 package org.apache.commons.math3.analysis.interpolation;
 
 import org.apache.commons.math3.exception.DimensionMismatchException;
+import org.apache.commons.math3.exception.InsufficientDataException;
 import org.apache.commons.math3.exception.MathIllegalArgumentException;
+import org.apache.commons.math3.exception.NonMonotonicSequenceException;
+import org.apache.commons.math3.exception.NullArgumentException;
 import org.apache.commons.math3.analysis.BivariateFunction;
 import org.apache.commons.math3.distribution.UniformRealDistribution;
 import org.apache.commons.math3.random.RandomGenerator;
@@ -27,53 +30,131 @@ import org.junit.Test;
 
 /**
  * Test case for the bicubic interpolator.
- * 
  */
-public final class BicubicSplineInterpolatorTest {
+public final class BicubicSplineInterpolatorTest
+{
     /**
      * Test preconditions.
      */
     @Test
-    public void testPreconditions() {
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2.5};
+    public void testPreconditions()
+    {
+        double[] xval = new double[] { 3, 4, 5, 6.5, 7.5 };
+        double[] yval = new double[] { -4, -3, -1, 2.5, 3.5 };
         double[][] zval = new double[xval.length][yval.length];
 
+        @SuppressWarnings( "unused" )
         BivariateGridInterpolator interpolator = new 
BicubicSplineInterpolator();
         
-        @SuppressWarnings("unused")
-        BivariateFunction p = interpolator.interpolate(xval, yval, zval);
+        try
+        {
+            interpolator.interpolate( null, yval, zval );
+            Assert.fail( "Failed to detect x null pointer" );
+        }
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            interpolator.interpolate( xval, null, zval );
+            Assert.fail( "Failed to detect y null pointer" );
+        }
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            interpolator.interpolate( xval, yval, null );
+            Assert.fail( "Failed to detect z null pointer" );
+        }
+        catch ( NullArgumentException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 2.0, 3.0 };
+            interpolator.interpolate( xval1, yval, zval );
+            Assert.fail( "Failed to detect insufficient x data" );
+        }
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 2.0, 3.0 };
+            interpolator.interpolate( xval, yval1, zval );
+            Assert.fail( "Failed to detect insufficient y data" );
+        }
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            double zval1[][] = new double[4][4];
+            interpolator.interpolate( xval, yval, zval1 );
+            Assert.fail( "Failed to detect insufficient z data" );
+        }
+        catch ( InsufficientDataException iae )
+        {
+            // Expected.
+        }
+
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0 };
+            interpolator.interpolate( xval1, yval, zval );
+            Assert.fail( "Failed to detect data set array with different 
sizes." );
+        }
+        catch ( DimensionMismatchException iae )
+        {
+            // Expected.
+        }
         
-        double[] wxval = new double[] {3, 2, 5, 6.5};
-        try {
-            p = interpolator.interpolate(wxval, yval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (MathIllegalArgumentException e) {
-            // Expected
-        }
-
-        double[] wyval = new double[] {-4, -3, -1, -1};
-        try {
-            p = interpolator.interpolate(xval, wyval, zval);
-            Assert.fail("an exception should have been thrown");
-        } catch (MathIllegalArgumentException e) {
-            // Expected
-        }
-
-        double[][] wzval = new double[xval.length][yval.length + 1];
-        try {
-            p = interpolator.interpolate(xval, yval, wzval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
-        }
-        wzval = new double[xval.length - 1][yval.length];
-        try {
-            p = interpolator.interpolate(xval, yval, wzval);
-            Assert.fail("an exception should have been thrown");
-        } catch (DimensionMismatchException e) {
-            // Expected
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0 };
+            interpolator.interpolate( xval, yval1, zval );
+            Assert.fail( "Failed to detect data set array with different 
sizes." );
+        }
+        catch ( DimensionMismatchException iae )
+        {
+            // Expected.
         }
+
+        // X values not sorted.
+        try
+        {
+            double xval1[] = { 0.0, 1.0, 0.5, 7.0, 3.5 };
+            interpolator.interpolate( xval1, yval, zval );
+            Assert.fail( "Failed to detect unsorted x arguments." );
+        }
+        catch ( NonMonotonicSequenceException iae )
+        {
+            // Expected.
+        }
+
+        // Y values not sorted.
+        try
+        {
+            double yval1[] = { 0.0, 1.0, 1.5, 0.0, 3.0 };
+            interpolator.interpolate( xval, yval1, zval );
+            Assert.fail( "Failed to detect unsorted y arguments." );
+        }
+        catch ( NonMonotonicSequenceException iae )
+        {
+            // Expected.
+        }
+
     }
 
     /**
@@ -82,26 +163,32 @@ public final class BicubicSplineInterpolatorTest {
      * z = 2 x - 3 y + 5
      */
     @Test
-    public void testInterpolation1() {
+    public void testInterpolation1()
+    {
         final int sz = 21;
         double[] xval = new double[sz];
         double[] yval = new double[sz];
         // Coordinate values
         final double delta = 1d / (sz - 1);
-        for (int i = 0; i < sz; i++) {
+        for ( int i = 0; i < sz; i++ )
+        {
             xval[i] = -1 + 15 * i * delta;
             yval[i] = -20 + 30 * i * delta;
         }
 
         // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
+        BivariateFunction f = new BivariateFunction()
+        {
+            public double value( double x, double y )
+            {
                     return 2 * x - 3 * y + 5;
                 }
             };
         double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
+        for ( int i = 0; i < xval.length; i++ )
+        {
+            for ( int j = 0; j < yval.length; j++ )
+            {
                 zval[i][j] = f.value(xval[i], yval[j]);
             }
         }
@@ -111,16 +198,16 @@ public final class BicubicSplineInterpolatorTest {
         double x, y;
 
         final RandomGenerator rng = new Well19937c(1234567L); // "tol" depends 
on the seed.
-        final UniformRealDistribution distX
-            = new UniformRealDistribution(rng, xval[0], xval[xval.length - 1]);
-        final UniformRealDistribution distY
-            = new UniformRealDistribution(rng, yval[0], yval[yval.length - 1]);
+        final UniformRealDistribution distX = new UniformRealDistribution( 
rng, xval[0], xval[xval.length - 1] );
+        final UniformRealDistribution distY = new UniformRealDistribution( 
rng, yval[0], yval[yval.length - 1] );
 
         final int numSamples = 50;
-        final double tol = 6;
-        for (int i = 0; i < numSamples; i++) {
+        final double tol = 2e-14;
+        for ( int i = 0; i < numSamples; i++ )
+        {
             x = distX.sample();
-            for (int j = 0; j < numSamples; j++) {
+            for ( int j = 0; j < numSamples; j++ )
+            {
                 y = distY.sample();
 //                 System.out.println(x + " " + y + " " + f.value(x, y) + " " 
+ p.value(x, y));
                 Assert.assertEquals(f.value(x, y),  p.value(x, y), tol);
@@ -135,26 +222,32 @@ public final class BicubicSplineInterpolatorTest {
      * z = 2 x<sup>2</sup> - 3 y<sup>2</sup> + 4 x y - 5
      */
     @Test
-    public void testInterpolation2() {
+    public void testInterpolation2()
+    {
         final int sz = 21;
         double[] xval = new double[sz];
         double[] yval = new double[sz];
         // Coordinate values
         final double delta = 1d / (sz - 1);
-        for (int i = 0; i < sz; i++) {
+        for ( int i = 0; i < sz; i++ )
+        {
             xval[i] = -1 + 15 * i * delta;
             yval[i] = -20 + 30 * i * delta;
         }
 
         // Function values
-        BivariateFunction f = new BivariateFunction() {
-                public double value(double x, double y) {
+        BivariateFunction f = new BivariateFunction()
+        {
+            public double value( double x, double y )
+            {
                     return 2 * x * x - 3 * y * y + 4 * x * y - 5;
                 }
             };
         double[][] zval = new double[xval.length][yval.length];
-        for (int i = 0; i < xval.length; i++) {
-            for (int j = 0; j < yval.length; j++) {
+        for ( int i = 0; i < xval.length; i++ )
+        {
+            for ( int j = 0; j < yval.length; j++ )
+            {
                 zval[i][j] = f.value(xval[i], yval[j]);
             }
         }
@@ -164,16 +257,16 @@ public final class BicubicSplineInterpolatorTest {
         double x, y;
 
         final RandomGenerator rng = new Well19937c(1234567L); // "tol" depends 
on the seed.
-        final UniformRealDistribution distX
-            = new UniformRealDistribution(rng, xval[0], xval[xval.length - 1]);
-        final UniformRealDistribution distY
-            = new UniformRealDistribution(rng, yval[0], yval[yval.length - 1]);
+        final UniformRealDistribution distX = new UniformRealDistribution( 
rng, xval[0], xval[xval.length - 1] );
+        final UniformRealDistribution distY = new UniformRealDistribution( 
rng, yval[0], yval[yval.length - 1] );
 
         final int numSamples = 50;
-        final double tol = 251;
-        for (int i = 0; i < numSamples; i++) {
+        final double tol = 5e-13;
+        for ( int i = 0; i < numSamples; i++ )
+        {
             x = distX.sample();
-            for (int j = 0; j < numSamples; j++) {
+            for ( int j = 0; j < numSamples; j++ )
+            {
                 y = distY.sample();
 //                 System.out.println(x + " " + y + " " + f.value(x, y) + " " 
+ p.value(x, y));
                 Assert.assertEquals(f.value(x, y),  p.value(x, y), tol);

http://git-wip-us.apache.org/repos/asf/commons-math/blob/d8bfc8c8/src/test/java/org/apache/commons/math3/analysis/interpolation/SmoothingPolynomialBicubicSplineInterpolatorTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/SmoothingPolynomialBicubicSplineInterpolatorTest.java
 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/SmoothingPolynomialBicubicSplineInterpolatorTest.java
index 32dbd2f..51c9760 100644
--- 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/SmoothingPolynomialBicubicSplineInterpolatorTest.java
+++ 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/SmoothingPolynomialBicubicSplineInterpolatorTest.java
@@ -33,8 +33,8 @@ public final class 
SmoothingPolynomialBicubicSplineInterpolatorTest {
      */
     @Test
     public void testPreconditions() {
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2.5};
+        double[] xval = new double[] {3, 4, 5, 6.5, 7.5};
+        double[] yval = new double[] {-4, -3, -1, 2.5, 3};
         double[][] zval = new double[xval.length][yval.length];
 
         BivariateGridInterpolator interpolator = new 
SmoothingPolynomialBicubicSplineInterpolator(0);
@@ -97,8 +97,8 @@ public final class 
SmoothingPolynomialBicubicSplineInterpolatorTest {
 
         BivariateGridInterpolator interpolator = new 
SmoothingPolynomialBicubicSplineInterpolator(1);
 
-        double[] xval = new double[] {3, 4, 5, 6.5};
-        double[] yval = new double[] {-4, -3, -1, 2, 2.5};
+        double[] xval = new double[] {3, 4, 5, 6.5, 7.5};
+        double[] yval = new double[] {-4, -3, -1, 2, 2.5, 3.5};
         double[][] zval = new double[xval.length][yval.length];
         for (int i = 0; i < xval.length; i++) {
             for (int j = 0; j < yval.length; j++) {
@@ -145,7 +145,7 @@ public final class 
SmoothingPolynomialBicubicSplineInterpolatorTest {
 
         BivariateGridInterpolator interpolator = new 
SmoothingPolynomialBicubicSplineInterpolator(4);
 
-        double[] xval = new double[] {3, 4, 5, 6.5};
+        double[] xval = new double[] {3, 4, 5, 6.5, 7.5, 8};
         double[] yval = new double[] {-4, -3, -2, -1, 0.5, 2.5};
         double[][] zval = new double[xval.length][yval.length];
         for (int i = 0; i < xval.length; i++) {

http://git-wip-us.apache.org/repos/asf/commons-math/blob/d8bfc8c8/src/test/java/org/apache/commons/math3/analysis/interpolation/SplineInterpolatorTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/SplineInterpolatorTest.java
 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/SplineInterpolatorTest.java
index 95ce8fa..f9b2e3e 100644
--- 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/SplineInterpolatorTest.java
+++ 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/SplineInterpolatorTest.java
@@ -34,17 +34,18 @@ import org.junit.Test;
 public class SplineInterpolatorTest {
 
     /** error tolerance for spline interpolator value at knot points */
-    protected double knotTolerance = 1E-12;
+    protected double knotTolerance = 1E-14;
 
     /** error tolerance for interpolating polynomial coefficients */
-    protected double coefficientTolerance = 1E-6;
+    protected double coefficientTolerance = 1E-14;
 
     /** error tolerance for interpolated values -- high value is from sin test 
*/
-    protected double interpolationTolerance = 1E-2;
+    protected double interpolationTolerance = 1E-14;
 
     @Test
     public void testInterpolateLinearDegenerateTwoSegment()
         {
+        double tolerance = 1e-15;
         double x[] = { 0.0, 0.5, 1.0 };
         double y[] = { 0.0, 0.5, 1.0 };
         UnivariateInterpolator i = new SplineInterpolator();
@@ -60,14 +61,15 @@ public class SplineInterpolatorTest {
         TestUtils.assertEquals(polynomials[1].getCoefficients(), target, 
coefficientTolerance);
 
         // Check interpolation
-        Assert.assertEquals(0.0,f.value(0.0), interpolationTolerance);
-        Assert.assertEquals(0.4,f.value(0.4), interpolationTolerance);
-        Assert.assertEquals(1.0,f.value(1.0), interpolationTolerance);
+        Assert.assertEquals(0.0,f.value(0.0), tolerance);
+        Assert.assertEquals(0.4,f.value(0.4), tolerance);
+        Assert.assertEquals(1.0,f.value(1.0), tolerance);
     }
 
     @Test
     public void testInterpolateLinearDegenerateThreeSegment()
         {
+        double tolerance = 1e-15;
         double x[] = { 0.0, 0.5, 1.0, 1.5 };
         double y[] = { 0.0, 0.5, 1.0, 1.5 };
         UnivariateInterpolator i = new SplineInterpolator();
@@ -84,9 +86,9 @@ public class SplineInterpolatorTest {
         TestUtils.assertEquals(polynomials[2].getCoefficients(), target, 
coefficientTolerance);
 
         // Check interpolation
-        Assert.assertEquals(0,f.value(0), interpolationTolerance);
-        Assert.assertEquals(1.4,f.value(1.4), interpolationTolerance);
-        Assert.assertEquals(1.5,f.value(1.5), interpolationTolerance);
+        Assert.assertEquals(0,f.value(0), tolerance);
+        Assert.assertEquals(1.4,f.value(1.4), tolerance);
+        Assert.assertEquals(1.5,f.value(1.5), tolerance);
     }
 
     @Test
@@ -108,6 +110,8 @@ public class SplineInterpolatorTest {
 
     @Test
     public void testInterpolateSin() {
+        double sineCoefficientTolerance = 1e-6;
+        double sineInterpolationTolerance = 0.0043;
         double x[] =
             {
                 0.0,
@@ -136,25 +140,25 @@ public class SplineInterpolatorTest {
          */
         PolynomialFunction polynomials[] = ((PolynomialSplineFunction) 
f).getPolynomials();
         double target[] = {y[0], 1.002676d, 0d, -0.17415829d};
-        TestUtils.assertEquals(polynomials[0].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[0].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[1], 8.594367e-01, -2.735672e-01, -0.08707914};
-        TestUtils.assertEquals(polynomials[1].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[1].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[2], 1.471804e-17,-5.471344e-01, 0.08707914};
-        TestUtils.assertEquals(polynomials[2].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[2].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[3], -8.594367e-01, -2.735672e-01, 0.17415829};
-        TestUtils.assertEquals(polynomials[3].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[3].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[4], -1.002676, 6.548562e-17, 0.17415829};
-        TestUtils.assertEquals(polynomials[4].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[4].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[5], -8.594367e-01, 2.735672e-01, 0.08707914};
-        TestUtils.assertEquals(polynomials[5].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[5].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[6], 3.466465e-16, 5.471344e-01, -0.08707914};
-        TestUtils.assertEquals(polynomials[6].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[6].getCoefficients(), target, 
sineCoefficientTolerance);
         target = new double[]{y[7], 8.594367e-01, 2.735672e-01, -0.17415829};
-        TestUtils.assertEquals(polynomials[7].getCoefficients(), target, 
coefficientTolerance);
+        TestUtils.assertEquals(polynomials[7].getCoefficients(), target, 
sineCoefficientTolerance);
 
         //Check interpolation
-        Assert.assertEquals(FastMath.sqrt(2d) / 
2d,f.value(FastMath.PI/4d),interpolationTolerance);
-        Assert.assertEquals(FastMath.sqrt(2d) / 
2d,f.value(3d*FastMath.PI/4d),interpolationTolerance);
+        Assert.assertEquals(FastMath.sqrt(2d) / 
2d,f.value(FastMath.PI/4d),sineInterpolationTolerance);
+        Assert.assertEquals(FastMath.sqrt(2d) / 
2d,f.value(3d*FastMath.PI/4d),sineInterpolationTolerance);
     }
 
     @Test

http://git-wip-us.apache.org/repos/asf/commons-math/blob/d8bfc8c8/src/test/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatorTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatorTest.java
 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatorTest.java
index 6de42a2..be5a865 100644
--- 
a/src/test/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatorTest.java
+++ 
b/src/test/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatorTest.java
@@ -32,7 +32,7 @@ public final class TricubicSplineInterpolatorTest {
     /**
      * Test preconditions.
      */
-    @Test
+    @Ignore@Test
     public void testPreconditions() {
         double[] xval = new double[] {3, 4, 5, 6.5};
         double[] yval = new double[] {-4, -3, -1, 2.5};

Reply via email to