Skip to content

Comments

40 implement hybrid search#59

Closed
Zafarhussain87 wants to merge 12 commits intomainfrom
40-implement-hybrid-search
Closed

40 implement hybrid search#59
Zafarhussain87 wants to merge 12 commits intomainfrom
40-implement-hybrid-search

Conversation

@Zafarhussain87
Copy link
Contributor

@Zafarhussain87 Zafarhussain87 commented Dec 16, 2025

Implements an advanced search method:

Hybrid Search: RRF fusion of semantic and FTS5 keyword search
semantic search works with article_embeddings_e5 index
FTS5 search works with fts_article_embeddings_e5 index

Features:
New API endpoint(hybrid_search) is added for the search method
New method in search_knowledge_base

Test:
run the beta_workflow_v1_large as given below to verify the hybrid_search method:
curl -X POST http://localhost:8001/workflows/beta_workflow_v1_large/run
-H "Content-Type: application/json"
-d '{
"user_input": "Hvad er Aasiaat og hvad er byens vigtigste erhverv?",
"conversation_id": "tc6-19n",
"conversation_history": []
}'

Implements an advanced search method:
- Hybrid Search: RRF fusion of semantic and FTS5 keyword search
- semantic search works with article_embeddings_e5 index
- FTS5 search works with fts_article_embeddings_e5 index

Features:
- New API endpoint(hybrid_search) is added for the search method
- New method in search_knowledge_base
Implements an advanced search method:
- Hybrid Search: RRF fusion of semantic and FTS5 keyword search
- semantic search works with article_embeddings_e5 index
- FTS5 search works with fts_article_embeddings_e5 index

Features:
- New API endpoint(hybrid_search) is added for the search method
- New method in search_knowledge_base
Corrected the search method from 'hybrid_hyde_search' to 'hybrid_search'.
Implements an advanced search method:
- Hybrid Search: RRF fusion of semantic and FTS5 keyword search
- semantic search works with article_embeddings_e5 index
- FTS5 search works with fts_article_embeddings_e5 index

Features:
- New API endpoint(hybrid_search) is added for the search method
- New method in search_knowledge_base
@Zafarhussain87 Zafarhussain87 linked an issue Dec 16, 2025 that may be closed by this pull request
Copy link
Contributor

@Enniwhere Enniwhere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only have a very small change requested: Make new workflows instead of changing the existing ones.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should avoid changing the existing workflows and rather make new ones, even if it means having more duplicate code.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement hybrid search

2 participants