Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Consolidate OpenAI providers #455

Open
andrii-i opened this issue Nov 9, 2023 · 2 comments
Open

Consolidate OpenAI providers #455

andrii-i opened this issue Nov 9, 2023 · 2 comments
Labels
enhancement New feature or request
Milestone
v3.0.0

Comments

@andrii-i
Copy link
Collaborator

andrii-i commented Nov 9, 2023

Problem

Currently there are 3 OpenAI providers: openai, openai-chat, openai-chat-new. This is confusing, the same service by the same company shouldn’t have 3 different provider IDs.

Proposed Solution

Ideally, consolidate all OpenAI providers into a single provider (for example, openai).

Additional context

Note that openai-chat:gpt-3.5-turbo and openai-chat-new:gpt-3.5-turbo is the same model.
@andrii-i andrii-i added the enhancement New feature or request label Nov 9, 2023
@JasonWeill
Copy link
Collaborator

To avoid breaking older notebooks, we should treat calls to openai-chat or openai-chat-new as calls to openai models after this is fixed.

@dlqqq
Copy link
Member

dlqqq commented Nov 27, 2024

We should explore allowing one model provider to refer to multiple LangChain classes in v3. Adding this issue to the milestone.

@dlqqq dlqqq added this to the v3.0.0 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
v3.0.0
Development

No branches or pull requests
3 participants
@andrii-i @dlqqq @JasonWeill