Skip to content

fix: infinite kmeans if the largest cluster produces only 1 cluster#5078

Merged
BubbleCal merged 5 commits intolance-format:mainfrom
BubbleCal:fix-kmeans-all-zeros
Oct 29, 2025
Merged

fix: infinite kmeans if the largest cluster produces only 1 cluster#5078
BubbleCal merged 5 commits intolance-format:mainfrom
BubbleCal:fix-kmeans-all-zeros

Conversation

@BubbleCal
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: BubbleCal <bubble-cal@outlook.com>
@github-actions github-actions Bot added the bug Something isn't working label Oct 27, 2025
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread rust/lance-index/src/vector/kmeans.rs Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 27, 2025

Codecov Report

❌ Patch coverage is 70.68966% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.69%. Comparing base (31ef40e) to head (8cee497).

Files with missing lines Patch % Lines
rust/lance-index/src/vector/kmeans.rs 70.68% 15 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5078      +/-   ##
==========================================
- Coverage   81.73%   81.69%   -0.04%     
==========================================
  Files         340      340              
  Lines      138845   138875      +30     
  Branches   138845   138875      +30     
==========================================
- Hits       113480   113460      -20     
- Misses      21626    21678      +52     
+ Partials     3739     3737       -2     
Flag Coverage Δ
unittests 81.69% <70.68%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: BubbleCal <bubble-cal@outlook.com>
Signed-off-by: BubbleCal <bubble-cal@outlook.com>
Signed-off-by: BubbleCal <bubble-cal@outlook.com>
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you!

@BubbleCal BubbleCal merged commit 65609d6 into lance-format:main Oct 29, 2025
27 checks passed
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants