[PR #17617] [CLOSED] FIX: hybrid_search-instances_checker #24491

New Issue

GiteaMirror · 2026-04-20T05:25:43-05:00

GiteaMirror commented

2026-04-20 05:25:43 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/17617
Author: @rgaricano
Created: 9/20/2025
Status: ❌ Closed

Base: dev ← Head: dev-FIX_hybrid_search-instances_checker

📝 Commits (1)

bf2e25f FIX: hibryd_search-instances_checker

📊 Changes

1 file changed (+10 additions, -2 deletions)

View changed files

📝 backend/open_webui/retrieval/utils.py (+10 -2)

📄 Description

FIX: hybrid_search-fails due to inconsistence in parameters type received in some circunstances.

In some circunstances hybrid search fail, as it's exposed in issue https://github.com/open-webui/open-webui/issues/17046

query_doc_with_hybrid_search function expects collection_result to be an object with a documents attribute, but it's receiving a list or None instead (e.g. qdrant return None if collections don't exist, which could cause the issue)
when query is as list (e.g. query_doc_with_hybrid_search function called for queries)

This PR add a couple of checkers for that.

Contributor License Agreement

By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/open-webui/pull/17617 **Author:** [@rgaricano](https://github.com/rgaricano) **Created:** 9/20/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `dev-FIX_hybrid_search-instances_checker` --- ### 📝 Commits (1) - [`bf2e25f`](https://github.com/open-webui/open-webui/commit/bf2e25fd1870ad7108f1e38be6db1986c5384474) FIX: hibryd_search-instances_checker ### 📊 Changes **1 file changed** (+10 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/retrieval/utils.py` (+10 -2) </details> ### 📄 Description ### FIX: hybrid_search-fails due to inconsistence in parameters type received in some circunstances. In some circunstances hybrid search fail, as it's exposed in issue https://github.com/open-webui/open-webui/issues/17046 - query_doc_with_hybrid_search function expects collection_result to be an object with a documents attribute, but it's receiving a list or None instead (e.g. qdrant return None if collections don't exist, which could cause the issue) - when query is as list (e.g. query_doc_with_hybrid_search function called for queries) This PR add a couple of checkers for that. ____ ### Contributor License Agreement By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-04-20 05:25:43 -05:00

GiteaMirror closed this issue