feat: add support for multi filter retriever#11028
feat: add support for multi filter retriever#11028srini047 wants to merge 9 commits intodeepset-ai:mainfrom
Conversation
|
@srini047 is attempting to deploy a commit to the deepset Team on Vercel. A member of the Team first needs to authorize it. |
bogdankostic
left a comment
There was a problem hiding this comment.
Hi @srini047, thank you very much for the PR! I left a few comments on how we can simplify the MultiFilterRetriever.
releasenotes/notes/feat-multi-filter-retriever-266283c61e693da3.yaml
Outdated
Show resolved
Hide resolved
|
Thanks for addressing the requested changes @srini047! Looking at the current implementation, since |
@bogdankostic If we look at the issue that was the ask to take filters as input: WDYT? |
Related Issues
MultiFilterRetriever#10995Proposed Changes:
Add support for
MultiFilterRetriever, a new retriever component that executes multiple filter queries against a document store in parallel and returns a single, de-duplicated list of documents sorted by relevance score.How did you test it?
Added unit tests around the new filter and ensured older doesn't break.
Notes for the reviewer
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:and added!in case the PR includes breaking changes.