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

Adds support for Azure OpenAI #2308

Closed
wants to merge 9 commits into from

Conversation

frandominguezl
Copy link

Pull Request Checklist

  • Target branch: Pull requests should target the dev branch.
  • Description: Briefly describe the changes in this pull request.
  • Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
  • Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
  • Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
  • Testing: Have you written and run sufficient tests for the changes?
  • Code Review: Have you self-reviewed your code and addressed any coding standard issues?

Description

Adds native support for Azure OpenAI, in a similar way that it's implemented for OpenAI, rather than relying in LiteLLM.


Changelog Entry

Added

  • Support for Azure OpenAI

@justinh-rahb
Copy link
Collaborator

Builds on Mac, but I haven't got an Azure API to test with.

# Conflicts:
#	backend/config.py
#	backend/main.py
#	src/routes/(app)/+page.svelte
#	src/routes/(app)/c/[id]/+page.svelte
@jannikstdl
Copy link
Contributor

This should be already possible with litellm, isn't it?

@tjbck
Copy link
Contributor

tjbck commented May 16, 2024

Ideally we'd want to integrate azure support to current existing openai codebase. I appreciate the effort!

Related #1667

@tjbck tjbck closed this May 16, 2024
@rcarmo
Copy link

rcarmo commented May 22, 2024

Can we get a resolution here? #1667 is also closed, and this would be an extremely useful feature.

@justinh-rahb
Copy link
Collaborator

Can we get a resolution here? #1667 is also closed, and this would be an extremely useful feature.

Closed for same reason: it was implemented as an entirely separate app within the backend when what Tim desires is for the existing openai app to be extended to support azure endpoints instead.

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.

None yet

5 participants