feat: Astra - improve filters and add unit tests#3194
Conversation
Coverage report (astra)Click to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||||||||||||||||||||
bogdankostic
left a comment
There was a problem hiding this comment.
Looking good in principle, I was just wondering if we could remove the overwrites of the filter tests below this line, given that filters seem to be supported looking at test_filters_unit.py.
Prompted by this comment, I decided to investigate better. And I realized that we could not remove overwrites right away. So I decided to implement better filters for Astra (with some help from AI) and now filters are more aligned with Haystack defaults. I also updated this PR description: #3194 (comment) |
Related Issues
Proposed Changes:
How did you test it?
CI
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:.