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

community: Chroma - remove reference to collection upon delete_collection #21817

Conversation

tazarov
Copy link
Contributor

@tazarov tazarov commented May 17, 2024

Description:

  • Reference to Collection object is set to None when deleting a collection delete_collection()
  • Added utility method reset_collection() to allow recreating the collection
  • Moved collection creation out of __init__ into __ensure_collection() to be reused by object init and reset_collection()
  • _collection is now a property to avoid breaking changes

Issues:

Twitter: @t_azarov

@efriis efriis added the partner label May 17, 2024
@efriis efriis self-assigned this May 17, 2024
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 17, 2024
Copy link

vercel bot commented May 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview May 17, 2024 3:11pm

@dosubot dosubot bot added Ɑ: vector store Related to vector store module 🔌: chroma Primarily related to ChromaDB integrations 🤖:improvement Medium size change to existing code to handle new use-cases labels May 17, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label May 17, 2024
@baskaryan baskaryan merged commit f54cbf8 into langchain-ai:master May 20, 2024
21 checks passed
JuHyung-Son pushed a commit to JuHyung-Son/langchain that referenced this pull request May 23, 2024
…llection (langchain-ai#21817)

**Description**:

- Reference to `Collection` object is set to `None` when deleting a
collection `delete_collection()`
- Added utility method `reset_collection()` to allow recreating the
collection
- Moved collection creation out of `__init__` into
`__ensure_collection()` to be reused by object init and
`reset_collection()`
- `_collection` is now a property to avoid breaking changes

**Issues**: 

- chroma-core/chroma#2213

**Twitter**: @t_azarov
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌: chroma Primarily related to ChromaDB integrations 🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. partner size:M This PR changes 30-99 lines, ignoring generated files. Ɑ: vector store Related to vector store module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants