[ 
https://issues.apache.org/jira/browse/LUCENE-7179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15226307#comment-15226307
 ] 

Robert Muir commented on LUCENE-7179:
-------------------------------------

stability is already separately tested, e.g.:

https://github.com/apache/lucene-solr/blob/master/lucene/sandbox/src/test/org/apache/lucene/document/TestLatLonPoint.java#L109

I think this change would make some of these tests very complex or impossible, 
as e.g. some of them put the value in a stored field / array /or similar for 
comparisons.

It is ok two quantize twice or even 10,000 times or there is a bug.

> GeoPoint and LatLonPoint test data should quantize once
> -------------------------------------------------------
>
>                 Key: LUCENE-7179
>                 URL: https://issues.apache.org/jira/browse/LUCENE-7179
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Nicholas Knize
>
> {{LatLonPoint}} and {{GeoPointField}} tests pre quantizes test data to ensure 
> consistency with indexed (encoded) data. The pre quantized data then becomes 
> indexed, undergoing another quantization. To guarantee numerical stability 
> this should be changed such that the test data is quantized after indexing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to