Repository navigation
CommunityToolkit.VectorData.Chroma 1.0.0-preview - #9
Merged
Merged
Conversation
…maDotNet.VectorData
There was a problem hiding this comment.
🟢 Approval recommended
The package, namespace, API usages, tests, and documentation are consistently migrated with no unresolved issues.
0 open findings
What changed in this PR
Migrates the samples and integration tests to the AI Community Toolkit’s Chroma provider.
Changes:
- Replaces
ChromaDotNet.VectorDatawithCommunityToolkit.VectorData.Chroma1.0.0-preview. - Updates namespaces across samples and tests.
- Updates the README package links and attribution.
| File | Description |
|---|---|
test/ChromaDB.SemanticKernel.Sample.Tests/MultiStoreTests.cs |
Uses the new provider namespace. |
test/ChromaDB.SemanticKernel.Sample.Tests/DataModelAndPagingTests.cs |
Uses the new provider namespace. |
test/ChromaDB.SemanticKernel.Sample.Tests/ChatCompletionRagTests.cs |
Uses the new provider namespace. |
samples/GettingStartedWithTextSearch/Step5_Search_With_Chroma.cs |
Migrates the Azure OpenAI sample namespace. |
samples/GettingStartedWithTextSearch/Step5_Search_With_Chroma_Ollama.cs |
Migrates the Ollama sample namespace. |
samples/GettingStartedWithTextSearch/GettingStartedWithTextSearch.csproj |
References the replacement package. |
samples/GettingStartedWithTextSearch/ChromaVectorStoreFixtureBase.cs |
Imports provider types from the new namespace. |
samples/Concepts/Memory/VectorStore_VectorSearch_Paging_Chroma.cs |
Migrates the paging sample namespace. |
samples/Concepts/Memory/VectorStore_VectorSearch_MultiStore_Chroma.cs |
Migrates the multi-store sample namespace. |
samples/Concepts/Memory/VectorStore_HybridSearch_Simple_Chroma.cs |
Migrates the hybrid-search sample namespace. |
samples/Concepts/Memory/VectorStore_DynamicDataModel_Interop_Chroma.cs |
Migrates the interop sample namespace. |
samples/Concepts/Concepts.csproj |
References the replacement package. |
samples/Concepts/Agents/ChatCompletion_Rag_Chroma.cs |
Migrates the Azure OpenAI RAG sample namespace. |
samples/Concepts/Agents/ChatCompletion_Rag_Chroma_Ollama.cs |
Migrates the Ollama RAG sample namespace. |
README.md |
Documents and links the new provider. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CommunityToolkit.VectorData.Chroma 1.0.0-preview in place of ChromaDotNet.VectorData 0.4.3.
The samples and the tests reference
CommunityToolkit.VectorData.Chroma1.0.0-preview from nuget.org.The namespace changes. The types and their API stay the same:
The READMEs link to the new package. The main README says that it is the Chroma provider of the AI Community Toolkit.
Tested:
dotnet test test/ChromaDB.SemanticKernel.Sample.Tests: 10/10.ChatCompletion_Rag_Chroma_Ollama(2 tests) andStep5_Search_With_Chroma_Ollama(3 tests): passed.VectorStore_VectorSearch_Paging_Chroma: the pages go through the 1,000 records in order.VectorStore_HybridSearch_Simple_Chromaon Chroma Cloud: passed with the package from the GitHub releasemevd/Chroma/v1.0.0-preview. Its DLLs are the same as in the package on nuget.org.Not tested: the samples that use Azure OpenAI (
VectorStore_VectorSearch_MultiStore_Chroma,VectorStore_DynamicDataModel_Interop_Chroma,ChatCompletion_Rag_Chroma,Step5_Search_With_Chroma).