search

run-llama / llama_index

LlamaIndex is a data framework for your LLM applications
https://docs.llamaindex.ai
MIT License
33.3k stars 4.66k forks source link

[Bug]: Something needs to be updated with chromadb #14361

Open startakovsky opened 1 week ago

startakovsky commented 1 week ago

Bug Description

https://github.com/chroma-core/chroma/issues/2407#issuecomment-2187494105

Version

Latest

Steps to Reproduce

import chromadb

Relevant Logs/Tracbacks

No response

logan-markewich commented 1 week ago

Install numpy < 2.0

We have numpy pinned in llama-index-core already, but only in latest versions.

logan-markewich commented 1 week ago

Not really a llama-index issue per say here

startakovsky commented 1 week ago

Well, with the latest version of chromadb things are fine. I think it's because chromadb is pinned. Guess there's a choice to keep it pinned to the new or older version of chromadb. How do you usually make that decision @logan-markewich ?

startakovsky commented 1 week ago

Reference: Looks like the latest version of chroma fixes this.

https://github.com/chroma-core/chroma/issues/2407#issuecomment-2187899426

startakovsky commented 1 week ago

I see, so the dependencies here are listed below, curious why I seem to end up at chroma 0.5.0 when installing from scratch with no preference. I will try to fix it by explicitly stating the version.

https://github.com/run-llama/llama_index/blob/962853b4039b8465a12dad745c12e769aadc3322/llama-index-integrations/vector_stores/llama-index-vector-stores-chroma/pyproject.toml#L32-L35