msokolov commented on PR #14708:
URL: https://github.com/apache/lucene/pull/14708#issuecomment-2916875650

   +1 to add support for full precision re-ranking. Have you considered writing 
a FullPrecisionVectorSimilaritySource as a separate class?  We like to avoid 
conditional logic on boolean parameters where possible. I don't know if there 
is really a need for a byte-flavored version either? For the Query support we 
can expect the query to be supplied with high precision. During indexing 
wouldn't we want to re-rank using non-quantized query vectors as well as 
full-precision document vectors?  Not sure if that can be solved using a 
DoubleValuesSource - we would probably need to bake that in to the 
codec/hnswsearcher etc.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to