Skip to content

fix(llm): Fix when use custom embedding providers.#339

Open
lx200916 wants to merge 3 commits intovolcengine:mainfrom
lx200916:main
Open

fix(llm): Fix when use custom embedding providers.#339
lx200916 wants to merge 3 commits intovolcengine:mainfrom
lx200916:main

Conversation

@lx200916
Copy link

@lx200916 lx200916 commented Feb 9, 2026

Description

This pull request updates the embedding request logic in opencontext/llm/llm_client.py to better handle different LLM providers. The main change is switching the order of provider checks and ensuring that multimodal embeddings are used only for the DOUBAO provider.

Embedding provider logic update:

  • Changed the provider check to prioritize LLMProvider.DOUBAO and use multimodal_embeddings only for this provider, while other providers (like OPENAI) continue to use the standard embedding API.
image

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.

1 participant