weaviate / typescript-client

Official Weaviate TypeScript Client
https://www.npmjs.com/package/weaviate-ts-client
BSD 3-Clause "New" or "Revised" License
57 stars 21 forks source link

Add support for target vectors in hybrid queries #114

Closed joshdschneider closed 3 months ago

joshdschneider commented 3 months ago

Fixes 113.

tsmith023 commented 3 months ago

Hi @joshdschneider, thanks for the PR on this one! Could you please add an integration test within the describe block here for your changes, cheers! 😁

tsmith023 commented 3 months ago

@joshdschneider, your test fails (in quite an interesting way). It would seem that hybrid, due to its mixed nature as bm25 + vector search with the fusion algo, performs some cutoff returning only one result for your query!

joshdschneider commented 3 months ago

@tsmith023 ah, I see. Sorry about that—fixed!