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

chore: Use AzureOpenAI calls for model listings and completions #2443

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kk-src
Copy link
Contributor

@kk-src kk-src commented Feb 18, 2025

Replaced custom code for Azure OpenAI interactions with Azure OpenAI SDK calls. Some tasks, like listing deployed models, remain custom until Entra ID auth is introduced for control plane access. This commit uses SDK for listing all available models and handling chat completions, along with related code cleanup.

Please describe the purpose of this pull request.
Clean up custom code to use Azure OpenAI SDK calls for model listings and completions.

How to test
Testing can be done by deploying a mode in Azure OpenAI and running Letta Agent against it.

Have you tested this PR?
Manually tested against Azure OpenAI after deploying gpt-4o-min (with #2442 & #2441 applied together)

Related issues or PRs
N/A

Is your PR over 500 lines of code?
N/A

Additional context
N/A

Replaced custom code for Azure OpenAI interactions with Azure OpenAI
SDK calls. Some tasks, like listing deployed models, remain custom
until Entra ID auth is introduced for control plane access. This commit
uses SDK for listing all available models and handling chat completions,
along with related code cleanup.
@cpacker cpacker requested a review from mattzh72 February 20, 2025 02:22
@kk-src
Copy link
Contributor Author

kk-src commented Feb 20, 2025

Thank you @cpacker. @mattzh72 - wondering if there is any early feedback?

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.

1 participant