Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Make endpoint configurable for OpenAI completions client#52530

Merged
eseliger merged 1 commit into
mainfrom
es/oai-url
May 30, 2023
Merged

Make endpoint configurable for OpenAI completions client#52530
eseliger merged 1 commit into
mainfrom
es/oai-url

Conversation

@eseliger

Copy link
Copy Markdown
Member

This makes it possible to switch to an alternative OpenAI compatible endpoint, in line with what we allow for anthropic as well.

Test plan

Manual test.

This makes it possible to switch to an alternative OpenAI compatible endpoint, in line with what we allow for anthropic as well.
@eseliger eseliger requested a review from a team May 26, 2023 22:27
@cla-bot cla-bot Bot added the cla-signed label May 26, 2023

@abeatrix abeatrix left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀

@eseliger eseliger merged commit 808a17b into main May 30, 2023
@eseliger eseliger deleted the es/oai-url branch May 30, 2023 10:30
github-actions Bot pushed a commit that referenced this pull request May 30, 2023
This makes it possible to switch to an alternative OpenAI compatible
endpoint, in line with what we allow for anthropic as well.

## Test plan

Manual test.

(cherry picked from commit 808a17b)
eseliger added a commit that referenced this pull request May 31, 2023
…nt (#52601)

This makes it possible to switch to an alternative OpenAI compatible
endpoint, in line with what we allow for anthropic as well.

## Test plan

Manual test. <br> Backport 808a17b from
#52530

Co-authored-by: Erik Seliger <erikseliger@me.com>
ErikaRS pushed a commit that referenced this pull request Jun 22, 2023
This makes it possible to switch to an alternative OpenAI compatible
endpoint, in line with what we allow for anthropic as well.

## Test plan

Manual test.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants