Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SOLR-12471: testLLPDecodeIsStableAndPrice -- raise threshold some more

This commit is contained in:
David Smiley 2018-06-21 09:29:27 -04:00
parent fabc408d3b
commit 8eb006e4ed
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
solr/core/src/java/org/apache/solr/schema/LatLonPointSpatialField.java
View File

@ -87,7 +87,7 @@ public class LatLonPointSpatialField extends AbstractSpatialFieldType implements
public static String decodeDocValueToString(long value) { public static String decodeDocValueToString(long value) {
final double latDouble = GeoEncodingUtils.decodeLatitude((int) (value >> 32)); final double latDouble = GeoEncodingUtils.decodeLatitude((int) (value >> 32));
final double lonDouble = GeoEncodingUtils.decodeLongitude((int) (value & 0xFFFFFFFFL)); final double lonDouble = GeoEncodingUtils.decodeLongitude((int) (value & 0xFFFFFFFFL));
// This # decimal places gets us close to our available precision to 1.37cm; we have a test for it. // This # decimal places gets us close to our available precision to 1.40cm; we have a test for it.
// CEILING round-trips (decode then re-encode then decode to get identical results). Others did not. It also // CEILING round-trips (decode then re-encode then decode to get identical results). Others did not. It also
// reverses the "floor" that occurred when we encoded. // reverses the "floor" that occurred when we encoded.
final int DECIMAL_PLACES = 7; final int DECIMAL_PLACES = 7;

3
solr/core/src/test/org/apache/solr/search/TestSolr4Spatial2.java
View File

@ -168,7 +168,8 @@ public class TestSolr4Spatial2 extends SolrTestCaseJ4 {
//max found by trial & error. If we used 8 decimal places then we could get down to 1.04cm accuracy but then we //max found by trial & error. If we used 8 decimal places then we could get down to 1.04cm accuracy but then we
// lose the ability to round-trip -- 40 would become 39.99999997 (ugh). // lose the ability to round-trip -- 40 would become 39.99999997 (ugh).
assertTrue("deltaCm too high: " + deltaCentimeters, deltaCentimeters <= 1.37); assertTrue("deltaCm too high: " + deltaCentimeters, deltaCentimeters < 1.40);
// Pt(x=105.29894270124083,y=-0.4371673760042398) to Pt(x=105.2989428,y=-0.4371673) is 1.38568
} }
@Test @Test