benwtrent commented on issue #12579: URL: https://github.com/apache/lucene/issues/12579#issuecomment-1762966693
@kaivalnp yes, `KnnCollector` should be used for something like this :). Glad its useful! One of the tricky things I can see is that its possible that the bottom layer entry point is so bad, that it itself isn't within the `traversial_similarity`. Consequently, recall will suffer abysmally. But, a quick overview of your PR shows that this type of logic fits pretty easily. I look forward to the performance and recall tests! -- 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
