Skip to content

Conversation

@KoLogicDev
Copy link
Contributor

Summary

This PR adds a base_url parameter to the embedding_factory function, allowing users to specify a custom API endpoint for the embedding model. This is particularly useful when working with OpenAI-compatible APIs hosted on private or self-hosted infrastructure.

Changes

  • Added base_url: Optional[str] = None to embedding_factory
  • Passed base_url to OpenAIEmbeddings initialization

Motivation

Enables greater flexibility when using embedding_factory, such as routing requests to non-default or local endpoints.

Notes

No breaking changes introduced. Default behavior remains unchanged when base_url is not provided.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 18, 2025
Copy link
Member

@anistark anistark left a comment

Choose a reason for hiding this comment

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

It's a good to have. Thanks for the PR @KoLogicDev 🎉

However, merged is blocked by conflicts. Please rebase and get those sorted.

@anistark anistark added the blocked Blocked on further action label Aug 28, 2025
@anistark
Copy link
Member

Taken up in #2303

@anistark anistark closed this Sep 23, 2025
anistark added a commit that referenced this pull request Sep 23, 2025
## Issue Link / Problem Description
<!-- Link to related issue or describe the problem this PR solves -->
- contd #2019

Co-authored-by: Stepan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked Blocked on further action size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants