Skip to content

Comments

Update utils.py#77

Merged
Classic298 merged 5 commits intoembedding-perffrom
claude/resolve-merge-conflicts-016HV39f8affMyZBB4oyZsYJ
Nov 19, 2025
Merged

Update utils.py#77
Classic298 merged 5 commits intoembedding-perffrom
claude/resolve-merge-conflicts-016HV39f8affMyZBB4oyZsYJ

Conversation

@Classic298
Copy link
Owner

No description provided.

Classic298 and others added 5 commits November 17, 2025 22:56
Resolved merge conflicts between embedding-async and dev branches:

1. backend/open_webui/retrieval/utils.py:
   - Kept async signature for query_doc_with_hybrid_search
   - Added get_enriched_texts function from dev branch

2. backend/open_webui/utils/middleware.py:
   - Used direct async call to get_sources_from_items
   - Removed ThreadPoolExecutor wrapper (not needed with async)
   - Embedding function is now fully async-compatible
@Classic298 Classic298 merged commit 00ef239 into embedding-perf Nov 19, 2025
@Classic298 Classic298 deleted the claude/resolve-merge-conflicts-016HV39f8affMyZBB4oyZsYJ branch November 19, 2025 19:03
Classic298 added a commit that referenced this pull request Nov 24, 2025
…#19296)

* Update utils.py (#77)

Co-authored-by: Claude <[email protected]>

* refactor: address code review feedback for embedding performance improvements (#92)

Co-authored-by: Claude <[email protected]>

* fix: prevent sentence transformers from blocking async event loop (#95)

Co-authored-by: Claude <[email protected]>

---------

Co-authored-by: Claude <[email protected]>
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.

2 participants