Skip to content
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

Documention on Weaviate does not discuss saving and reloading later. Getting index_name/text_key errors when i test. #197

Open
heyjohnlim opened this issue Jul 7, 2024 · 2 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers

Comments

@heyjohnlim
Copy link

heyjohnlim commented Jul 7, 2024

The weaviate documentation does not explain how to save a database with an index_name nor load an existing vector database and query it. It assumes that we are just using from_documents or from_text.

Also when i was trying to load an existing vector database, i got errors saying index_name and text_key were required parameters to WeaviateVectorStore

Reference: https://python.langchain.com/v0.2/docs/integrations/vectorstores/weaviate/

Thanks again!

@heyjohnlim heyjohnlim added the enhancement New feature or request label Jul 7, 2024
@heyjohnlim heyjohnlim changed the title Documention on Weaviate Documention on Weaviate does not discuss saving and reloading later. Getting index_name/text_key errors when i test. Jul 7, 2024
@hsm207 hsm207 added documentation Improvements or additions to documentation and removed enhancement New feature or request labels Jul 7, 2024
@hsm207
Copy link
Collaborator

hsm207 commented Jul 7, 2024

It seems that based on issue langchain-ai/langchain#11471, the vector store interface does not specify a standard way to reuse an index and it is up to the implementation to define it.

@efriis can you confirm if this is true?

@efriis
Copy link
Member

efriis commented Jul 8, 2024

the parameters are integration-specific (e.g. weaviate has different parameters required to connect vs. chroma), and I would recommend adding documentation on how to connect to an existing store since it's not there yet! Maybe add it to the "connecting to weaviate" section?

Good issue!

@hsm207 hsm207 added enhancement New feature or request good first issue Good for newcomers labels Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants