Closed itayB closed 1 week ago
Please leverage this checklist to ensure your code review is thorough before approving
I notice that the import error here is from opt/hostedtoolcache/
- could this be a caching error unrelated to Chroma?
I notice that the import error here is from
opt/hostedtoolcache/
- could this be a caching error unrelated to Chroma?
Chroma is using tenacity
here: https://github.com/chroma-core/chroma/blob/main/chromadb/utils/embedding_functions.py#L5
The reason why you didn't see it - is because they already released another (fix) version:
Good catch. I have the feeling that since so many modules are numpy
dependent, we might have to freeze versions for the whole package to avoid a cascade of failures for our users.
If this happens a third time in the next 48 hours, I'll take a pass at that myself.
Sorry, my fault for putting the wrong title prefix on the PR!
I think this may no longer be required with the 8.4.1 release. I think tenacity 8.4.0 was just a botched release.
Confirmed 8.4.1 works - I am going to close this out but please let me know if I am doing to incorrectly @itayB
…s.txt
Description of changes
Summarize the changes made by this PR.
Test plan
How are these changes tested?
pytest
for python,yarn test
for js,cargo test
for rustDocumentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?