-
Notifications
You must be signed in to change notification settings - Fork 6
feat: Add support for remote Chroma database #58
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for remote Chroma database connections, migrating from local/self-hosted setups to cloud and enterprise deployments. The changes modernize the Chroma integration to work with newer versions and authentication methods.
- Updated Chroma client configuration to support remote instances with API tokens, tenants, and databases
- Replaced legacy authentication mechanism with modern API token-based authentication
- Updated dependencies to newer versions (ChromaDB 1.0.17, langchain-chroma 0.2.5, etc.)
Reviewed Changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docker-compose.yaml | Updates ChromaDB and OpenSearch images to specific newer versions |
| code/src/vector_stores/chroma.py | Refactors client initialization to support remote connections with API tokens, tenants, and databases |
| code/src/models/chroma_input_model.py | Updates input model to replace legacy auth fields with modern API token and tenant/database fields |
| code/tests/conftest.py | Updates test configuration to use new authentication parameters |
| code/src/utils.py | Updates import from langchain_community to langchain_apify |
| code/pyproject.toml | Updates dependencies to newer versions |
| code/.env.example | Updates environment variables for new authentication approach |
| actors/chroma/README.md | Updates documentation to reflect cloud/enterprise usage patterns |
| actors/chroma/.actor/input_schema.json | Updates input schema to match new authentication model |
| code/src/examples/*.py | Updates example scripts to use new configuration approach |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
MQ37
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did not test the integration, just scouted through the code. Just one change and question othewise LGTM 👍 Ready to merge after you address these.
Fixed, thanks! |
It was supposed to be a quick fix (and it actually was), but the integration and testing ended up taking longer than expected. 🤷🏻♂️
close: https://github.com/apify/ai-team/issues/62
FYI @MichalKalita