Skip to content
This repository was archived by the owner on Oct 9, 2026. It is now read-only.

Say in the README that Chroma 1.0.0 to 1.0.5 do not report the space of a collection - #20

Merged
Laurianti merged 1 commit into
mainfrom
readme-space-1.0.5
Oct 8, 2026
Merged

Laurianti merged 1 commit into
mainfrom
readme-space-1.0.5

Conversation

@Laurianti

Copy link
Copy Markdown
Member

The README of the package says that Chroma 1.0.0 to 1.0.5 do not report the space of a collection, so the check of the space cannot be done there, and suggests Chroma 1.0.6 or later. On those versions a collection created without a space uses l2, and a cosine search over it gives the scores of the l2 distances without an error; on Chroma 1.0.6 and on 0.6.3 the provider throws, as the README says.

No change of code or version.

@Laurianti
Laurianti requested a review from a team as a code owner October 8, 2026 11:05
Copilot AI balanced review requested due to automatic review settings October 8, 2026 11:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The documentation-only change clearly describes the limitation and recommended version.

0 open findings

What changed in this PR

Documents a Chroma server compatibility limitation affecting collection-space validation.

Changes:

  • Notes that Chroma 1.0.0–1.0.5 omit collection space metadata.
  • Recommends Chroma 1.0.6 or later.
File Description
src/​ChromaDB.VectorData/​README.md Adds the version-specific compatibility guidance.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

@Laurianti
Laurianti merged commit 19ef590 into main Oct 8, 2026
12 checks passed
@Laurianti
Laurianti deleted the readme-space-1.0.5 branch October 9, 2026 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants