Skip to content

[Backport 8.15] feat: extend default settings in AsyncVectorStore #2610

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Jul 4, 2024

Backport beb03de from #2602.

* feat: extend default settings in AsyncVectorStore

This commit introduces a new parameter, custom_settings, to the AsyncVectorStore class. This allows users to provide their own settings that will extend the default settings. This increases the flexibility of the class and allows it to be tailored to specific use cases. The custom settings are applied in the _create_index_if_not_exists method.

* Update vectorstore.py

* Update vectorstore.py

apply changes in vectorstore

* Update vectorstore.py

format the py file

* Update test_vectorstore.py

add test_custom_index_settings in test_vectorstore

* Update test_vectorstore.py

* Update vectorstore.py

fix file format

* Update test_vectorstore.py

fix format

* Update vectorstore.py

add error tips in vectorstore  when confilicting the settings

* Update vectorstore.py

* Update vectorstore.py

modify the comments of the param custom_index_settings

* Update vectorstore.py

* add settings conflict test

* reformat

---------

Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
Co-authored-by: Miguel Grinberg <miguel.grinberg@gmail.com>
(cherry picked from commit beb03de)
@miguelgrinberg miguelgrinberg reopened this Jul 4, 2024
@miguelgrinberg miguelgrinberg merged commit 10d7e69 into 8.15 Jul 4, 2024
12 checks passed
@miguelgrinberg miguelgrinberg deleted the backport-2602-to-8.15 branch July 4, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants