New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable hybrid search #794

Merged

PhilippeMoussalli merged 1 commit into main from enable-hybrid-search

Jan 19, 2024

Contributor

PhilippeMoussalli commented Jan 18, 2024

Fixes ml6team/fondant-usecase-RAG#70

No need to specify the vectorizer and it's module (needed for embedding the hybrid search query), it will automatically use the one specified in the indexing component based on the initial schema.
Tested the functionality indirectly,
Small fixes to default args of the indexing component


enable hybrid search

d3556d5

PhilippeMoussalli requested a review from RobbeSneyders

January 18, 2024 10:35

RobbeSneyders approved these changes

View reviewed changes

Member

RobbeSneyders left a comment

Lots of arguments 😅

PhilippeMoussalli merged commit b402b32 into main

9 checks passed

PhilippeMoussalli deleted the enable-hybrid-search branch

January 19, 2024 07:42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment