Skip to content

Support IsAzure option of ChatGpt LLM by AITuber controller#413

Merged
uezo merged 1 commit intouezo:masterfrom
buchizo:support-azure-openai-from-aituber-controller
Jan 14, 2025
Merged

Support IsAzure option of ChatGpt LLM by AITuber controller#413
uezo merged 1 commit intouezo:masterfrom
buchizo:support-azure-openai-from-aituber-controller

Conversation

@buchizo
Copy link
Contributor

@buchizo buchizo commented Jan 14, 2025

Currently, AITuber controller cannot configure LLM of the Azure OpenAI Service. The ChatGPT LLM Service has an IsAzure option, This PR can probably configure IsAzure option from the Controller using is_azure argument.

@uezo
Copy link
Owner

uezo commented Jan 14, 2025

Hi @buchizo ,
Thank you for your wonderful suggestion and for also submitting the pull request to chatdollkit-aituber! ✨

As you mentioned, there’s currently no way to configure IsAzure via the Controller API 😢.
While the idea of passing an is_azure argument is cool, and we may need to revisit this later to align with a more generalized design approach, I understand that this is an important and necessary change for now.

With that in mind, I’ll go ahead and merge your pull request! 🚀
In the future, we can revisit this and explore alternative approaches, such as the azure/ prefix idea inspired by LiteLLM.
https://docs.litellm.ai/docs/providers/azure

Thank you again for contributing to this project! Your input is truly appreciated.🙏

@uezo uezo merged commit cb5f119 into uezo:master Jan 14, 2025
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.

2 participants