Skip to content

MB-62182: bug fixes in the fast merge path#2302

Merged
abhinavdangeti merged 4 commits intomasterfrom
fixes
Apr 3, 2026
Merged

MB-62182: bug fixes in the fast merge path#2302
abhinavdangeti merged 4 commits intomasterfrom
fixes

Conversation

@Thejas-bhat
Copy link
Copy Markdown
Member

@Thejas-bhat Thejas-bhat commented Mar 30, 2026

  • introduce stats to track the number of samples the index was trained upon and the total training time.
  • guard the Batch() API in case the index's training is in progress.
  • exit training loop after the training is marked to be complete.
  • code cleanup

@Thejas-bhat Thejas-bhat marked this pull request as draft March 30, 2026 23:02
@abhinavdangeti
Copy link
Copy Markdown
Member

@Thejas-bhat can you expand what these bug fixes are - we ought to expedite this if the feature is incomplete without.

@abhinavdangeti abhinavdangeti added this to the v2.6.0 milestone Apr 1, 2026
@abhinavdangeti abhinavdangeti changed the title (draft) bug fixes in the fast merge path (draft) MB-62182: bug fixes in the fast merge path Apr 1, 2026
@Thejas-bhat Thejas-bhat changed the title (draft) MB-62182: bug fixes in the fast merge path MB-62182: bug fixes in the fast merge path Apr 2, 2026
@Thejas-bhat Thejas-bhat marked this pull request as ready for review April 2, 2026 22:33
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 2, 2026

Coverage Status

coverage: 52.532%. remained the same
when pulling e39dc61 on fixes
into deaa384 on master.

@abhinavdangeti abhinavdangeti merged commit 66904bb into master Apr 3, 2026
10 checks passed
@abhinavdangeti abhinavdangeti deleted the fixes branch April 3, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants