Skip to content

feat(namespace): add search_many - #258

Merged
Kaap10 merged 1 commit into
codeforstartups:developmentfrom
arshsk16:feat/namespace-search-many
Sep 26, 2026
Merged

Kaap10 merged 1 commit into
codeforstartups:developmentfrom
arshsk16:feat/namespace-search-many

Conversation

@arshsk16

Copy link
Copy Markdown
Contributor

Adds NamespaceView.search_many() to expose batched concurrent search while keeping queries pinned to the namespace.

Tests cover:

  • multiple queries with namespace isolation
  • forwarding top_k and filter kwargs
  • empty query batches

All local test pass

Fixes #244

This change was prepared with AI assistance under human direction and review.

@Kaap10 Kaap10 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM! Great work @arshsk16. The search_many implementation on NamespaceView is clean and test_namespace.py covers all edge cases nicely.

@Kaap10
Kaap10 merged commit acfd80e into codeforstartups:development Sep 26, 2026
3 checks passed
@arshsk16
arshsk16 deleted the feat/namespace-search-many branch September 26, 2026 15:19
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.

feat(api): Expose search_many() on NamespaceView

2 participants