Re: [PR] Introduce multiSelect for ScalarQuantizer [lucene]

2025-02-17 Thread via GitHub
HoustonPutman closed pull request #13919: Introduce multiSelect for ScalarQuantizer URL: https://github.com/apache/lucene/pull/13919 -- 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 comm

Re: [PR] Introduce multiSelect for ScalarQuantizer [lucene]

2024-10-30 Thread via GitHub
github-actions[bot] commented on PR #13919: URL: https://github.com/apache/lucene/pull/13919#issuecomment-2448726887 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Introduce multiSelect for ScalarQuantizer [lucene]

2024-10-16 Thread via GitHub
benwtrent commented on code in PR #13919: URL: https://github.com/apache/lucene/pull/13919#discussion_r1803334241 ## lucene/core/src/java/org/apache/lucene/util/quantization/ScalarQuantizer.java: ## @@ -568,29 +568,34 @@ private static List findNearestNeighbors( * and `95`.

Re: [PR] Introduce multiSelect for ScalarQuantizer [lucene]

2024-10-15 Thread via GitHub
HoustonPutman commented on PR #13919: URL: https://github.com/apache/lucene/pull/13919#issuecomment-2415275109 So after fixing the innocuous bugs in the implementations, it looks like there is no speed up here. The confidence interval finding can be up to 30% faster or so, but that's such a

[PR] Introduce multiSelect for ScalarQuantizer [lucene]

2024-10-15 Thread via GitHub
HoustonPutman opened a new pull request, #13919: URL: https://github.com/apache/lucene/pull/13919 Resolves #13918 ### Description This introduces a `multiSelect(from, to, k[])` method on the `Selector` abstract class, and gives implementations of the method for both `Selector`