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

[Bug]: Unable to load and create vector data base by using langchain module hugging face embedding #1952

Open
srnsrn120 opened this issue Apr 1, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@srnsrn120
Copy link

What happened?

from langchaincommunity.embeddings import HuggingFaceEmbeddings
embeddings = HuggingFaceEmbeddings(modelname="BAAI/bge-large-en-v1.5")
vectorstore_2 = Chroma(
collection_name="full_document", embedding_function=embeddings,persist_directory="./multi_vector_retriver_new"
)

ValueError: Expected EmbeddingFunction.__call to have the following signature: odict_keys(['self', 'input']), got odict_keys(['args', 'kwargs'])
Please see https://docs.trychroma.com/embeddings for details of the EmbeddingFunction interface.
Please note the recent change to the EmbeddingFunction interface: https://docs.trychroma.com/migration#migration-to-0416---november-7-2023

Hi team, i was able get response in morning but from 12 oclock i am getting this error , can anyone please help me

Versions

langchainhub==0.1.15
chromadb==0.4.24

Relevant log output

ValueError: Expected EmbeddingFunction.__call to have the following signature: odict_keys(['self', 'input']), got odict_keys(['args', 'kwargs'])
Please see https://docs.trychroma.com/embeddings for details of the EmbeddingFunction interface.
Please note the recent change to the EmbeddingFunction interface: https://docs.trychroma.com/migration#migration-to-0416---november-7-2023
@srnsrn120 srnsrn120 added the bug Something isn't working label Apr 1, 2024
@HammadB
Copy link
Collaborator

HammadB commented Apr 2, 2024

This should be fixed in our next release this coming monday - we now wrap the EF langchain gives us.

This PR needs to land - #1880

@tazarov
Copy link
Contributor

tazarov commented Apr 8, 2024

@srnsrn120, can you test your use case with the latest main branch and let us know if the new LC wrapper works for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants