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.

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

migrate models #3848

Merged

Conversation

lspinheiro
Copy link
Collaborator

Why are these changes needed?

Migrates openai models to autogen_ext to keep optional dependencies separate. Current models are kept with a FutureWarning.

Related issue number

Replaces #3781
Closes #3740

Checks

@lspinheiro lspinheiro requested a review from ekzhu October 21, 2024 00:38
@rysweet rysweet added the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 21, 2024
@jackgerrits jackgerrits merged commit 38f62e1 into microsoft:main Oct 22, 2024
30 checks passed
lspinheiro added a commit to lspinheiro/autogen that referenced this pull request Oct 23, 2024
* migrate models

* Update python/packages/autogen-agentchat/src/autogen_agentchat/agents/_tool_use_assistant_agent.py

Co-authored-by: Eric Zhu <[email protected]>

* refactor missing imports

* ignore type check errors

* Update python/packages/autogen-ext/src/autogen_ext/models/_openai/_model_info.py

Co-authored-by: Eric Zhu <[email protected]>

* update packages index page

---------

Co-authored-by: Leonardo Pinheiro <[email protected]>
Co-authored-by: Eric Zhu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster proj-extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Moving OpenAIChatCompletionClient and AzureOpenAIChatCompletionClient to autogen_ext
5 participants