Skip to content

[OMEGA-491] Allow to setting an embedding provider via the launcher - #387

Open
paul-v-snet wants to merge 3 commits into
patch-to-v0.1.20from
omega-491-asicloud-embedding-provider-fix
Open

paul-v-snet wants to merge 3 commits into
patch-to-v0.1.20from
omega-491-asicloud-embedding-provider-fix

Conversation

@paul-v-snet

Copy link
Copy Markdown
Collaborator

Description

Introduces a new -e parameter to start launcher command which allows to set an embedding provider when starting Omega.
The default embedding provider Local for the ASICloud provider was kept as before to ensure this won't break existing users vector stores after the update.

The parameter accepts only Local, OpenAI, and ASICloud; these are the complete set of currently supported embedding providers. The parameter is also case-sensitive since the value is compared with Local in src/memory.metta, and using local could lead to crashes:

(if (== (embeddingprovider) Local)

The final embedding provider is selected in the following order:

  1. -e <provider>
  2. EMBEDDING_PROVIDER if -e is not passed
  3. The default provider (Local for ASICloud) if neither of the above is provided

How Has This Been Tested?

  • Ran via start with the -e parameter and different providers: all passed
  • Ran mock tests: all green

Checklist

  • PR contains autogenerated code
  • Self-review completed
  • Test scenarios above are passed with the version of the code from PR

@timur-ashkenov timur-ashkenov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@paul-v-snet And if you choose different providers for the LLM and embeddings—for example, -p Anthropic -e OpenAI—how does the OpenAI key get into the container? Right now, it seems only the LLM provider's key is passed through. Is a separate key required for embeddings?

@paul-v-snet

Copy link
Copy Markdown
Collaborator Author

@timur-ashkenov, right, my bad, thanks for catching that. I'll fix it.

@TossSky TossSky closed this Oct 9, 2026
@TossSky TossSky reopened this Oct 9, 2026
@paul-v-snet

Copy link
Copy Markdown
Collaborator Author

@timur-ashkenov, fixed in ca3d3b5

This branch has not been deployed

No deployments
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.

3 participants