Skip to content

Fix bugs with SearchWithSelector#56

Merged
abhinavdangeti merged 2 commits intomasterfrom
bugFix
Apr 15, 2026
Merged

Fix bugs with SearchWithSelector#56
abhinavdangeti merged 2 commits intomasterfrom
bugFix

Conversation

@CascadingRadium
Copy link
Copy Markdown
Member

@CascadingRadium CascadingRadium commented Apr 15, 2026

  • Fix SearchWithSelector API signatures for float32 and BIVF indices.
  • Rename the API to SearchWithOptions which takes in optional constraints like:
    • Vector Selector
    • Search Parameters
  • If none of the constraints are provided, we fall back to the regular search API.
  • If either or both constraints are provided, we create a search params object and execute the FAISS API for parameterized search.

Thejas-bhat
Thejas-bhat reviewed Apr 15, 2026
View reviewed changes
Comment thread index.go
Thejas-bhat
Thejas-bhat reviewed Apr 15, 2026
View reviewed changes
Comment thread index.go Outdated
@abhinavdangeti abhinavdangeti merged commit 9d6f150 into master Apr 15, 2026
@abhinavdangeti abhinavdangeti deleted the bugFix branch April 15, 2026 21:37
@abhinavdangeti
Copy link
Copy Markdown
Member

abhinavdangeti commented Apr 15, 2026

v1.0.34

abhinavdangeti added a commit to blevesearch/bleve that referenced this pull request Apr 16, 2026
@abhinavdangeti
Absorb fixes for filtering vector search, update workflows
f084ace 
- blevesearch/go-faiss#56
- blevesearch/zapx#397
- blevesearch/zapx#398
@abhinavdangeti abhinavdangeti mentioned this pull request Apr 16, 2026
Merged
abhinavdangeti added a commit to blevesearch/bleve that referenced this pull request Apr 16, 2026
@abhinavdangeti
Absorb fixes for filtering vector search, update workflows (#2314)
936ca1f 
- blevesearch/go-faiss#56
- blevesearch/zapx#397
- blevesearch/zapx#398
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abhinavdangeti abhinavdangeti abhinavdangeti approved these changes

@Thejas-bhat Thejas-bhat Thejas-bhat approved these changes

@Likith101 Likith101 Awaiting requested review from Likith101

@capemox capemox Awaiting requested review from capemox

@maneuvertomars maneuvertomars Awaiting requested review from maneuvertomars

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@CascadingRadium @abhinavdangeti @Thejas-bhat