search

chroma-core / chroma

the AI-native open-source embedding database
https://www.trychroma.com/
Apache License 2.0
13.4k stars 1.14k forks source link

[Bug]: HYPOTHESIS does not include in DockerImage #2367

Closed cin-logan closed 1 week ago

cin-logan commented 1 week ago

What happened?

Hi, I have implemented Basic RBAC on Chroma following this documentation https://docs.trychroma.com/deployment/auth.

However, when I deployed, I received an error stating No module named 'hypothesis' when exporting the environment variable CHROMA_SERVER_AUTHZ_PROVIDER="chromadb.auth.simple_rbac_authz.SimpleRBACAuthorizationProvider". I checked the Docker image and Dockerfile and found that they don't include the Hypothesis library. It's only present in requirements_dev.txt.

Versions

Chroma0.5.0, Python3.11

Relevant log output

No response

tazarov commented 1 week ago

@cin-logan, thanks for reporting this. You are right that the RBAC Provider does indeed include some test artifacts:

https://github.com/chroma-core/chroma/blob/ec2e717a2d20f29db73eaae7d85e5f99ef7a810e/chromadb/auth/simple_rbac_authz/__init__.py#L19-L21

I believe these were added to fix slow test executions. We'll look into how to remove the dependency.

codetheweb commented 1 week ago

This is now fixed in 0.5.3. Thanks for reporting!