opensearch-project / k-NN

🆕 Find the k-nearest neighbors (k-NN) for your vector data
https://opensearch.org/docs/latest/search-plugins/knn/index/
Apache License 2.0
156 stars 123 forks source link

Introduce mode and compression param resolution #2034

Closed jmazanec15 closed 2 months ago

jmazanec15 commented 2 months ago

Description

Adds mode and compression based parameter resolution. With this, if a user specifies the mode and/or compression params, we will create a default configuration with the aim of meeting those hints.

Currently, it does not contain support for overriding any of the parameters. This will be taken in a future commit.

Still working on tests, but figured it would be good to get the PR out. Heres what I still need to do:

Related Issues

1949

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

navneet1v commented 2 months ago

Code looks good to me. Waiting for CI to be successful