You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2016/01/17 05:40:30 UTC

svn commit: r1725057 - /lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java

Author: dsmiley
Date: Sun Jan 17 04:40:30 2016
New Revision: 1725057

URL: http://svn.apache.org/viewvc?rev=1725057&view=rev
Log:
LUCENE-6873: Fix numerical precision issue in SpatialTestCase.randomGaussianMeanMax

Modified:
    lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java

Modified: lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java?rev=1725057&r1=1725056&r2=1725057&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java (original)
+++ lucene/dev/trunk/lucene/spatial/src/test/org/apache/lucene/spatial/SpatialTestCase.java Sun Jan 17 04:40:30 2016
@@ -227,7 +227,8 @@ public abstract class SpatialTestCase ex
     else
       pivotResult = Math.min(pivotMax, (g - 1) * (pivotMax - pivot) + pivot);
 
-    return mean + flip * pivotResult;
+    double result = mean + flip * pivotResult;
+    return (result < 0 || result > max) ? mean : result; // due this due to computational numerical precision
   }
 
   // ================================================= Inner Classes =================================================