This is an automated email from the ASF dual-hosted git repository.

erans pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-math.git

commit 78476b09dc530b3726951651efab44d952a348f2
Author: Gilles Sadowski <gillese...@gmail.com>
AuthorDate: Sun Jul 11 17:35:24 2021 +0200

    MATH-1462: Following advice in the Javadoc (on selecting the number of 
bins), unit test passes.
---
 .../math4/legacy/distribution/EmpiricalDistributionTest.java        | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/commons-math-legacy/src/test/java/org/apache/commons/math4/legacy/distribution/EmpiricalDistributionTest.java
 
b/commons-math-legacy/src/test/java/org/apache/commons/math4/legacy/distribution/EmpiricalDistributionTest.java
index 3705d6e..85a6f5c 100644
--- 
a/commons-math-legacy/src/test/java/org/apache/commons/math4/legacy/distribution/EmpiricalDistributionTest.java
+++ 
b/commons-math-legacy/src/test/java/org/apache/commons/math4/legacy/distribution/EmpiricalDistributionTest.java
@@ -667,7 +667,7 @@ public final class EmpiricalDistributionTest extends 
RealDistributionAbstractTes
         }
     }
 
-    @Ignore@Test
+    @Test
     public void testMath1462() {
         final double[] data = {
             6464.0205, 6449.1328, 6489.4569, 6497.5533, 6251.6487,
@@ -685,7 +685,7 @@ public final class EmpiricalDistributionTest extends 
RealDistributionAbstractTes
             6461.3944, 6384.1345
         };
 
-        final EmpiricalDistribution ed = new 
EmpiricalDistribution(data.length);
+        final EmpiricalDistribution ed = new EmpiricalDistribution(data.length 
/ 10);
         ed.load(data);
 
         double v;
@@ -693,7 +693,7 @@ public final class EmpiricalDistributionTest extends 
RealDistributionAbstractTes
 
         p = 0.49999;
         v = ed.inverseCumulativeProbability(p);
-        Assert.assertTrue("p=" + p + " => v=" + v, v < 6341);
+        Assert.assertTrue("p=" + p + " => v=" + v, v < 6344);
 
         p = 0.5;
         v = ed.inverseCumulativeProbability(p);

Reply via email to