-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Azure OpenAI Integration #690
Comments
@thickmn - thanks - we are working on it! |
@thinkmn - we have merged a proposed fix into the main branch - could you check it out and confirm if this will meet your needs in terms of configuration of the AzureOpenAI client - please check out the example - appreciate your help and support to test and make sure it meets your needs! |
@doberst - thanks for the quick response here! My first go at testing looks like it tries to call openai with a user managed api key and can't successfully set the azure client with the environment variables in the example. I'm going to spend more time on this tomorrow, but just as an FYI, when using langchain's integration, I had to pass: |
Hi, I had the same problem as thickmn, My GPT is the 3.5-turbo and to call my model is test: @doberst can you help me, please? |
@Clima2024 - happy to help with this - and sorry that you have run into an issue ... without sharing any confidential information, could you share the key 2-3 line code snippet, including the OpenAI configs... We will also test from our Azure account in parallel.... |
Thanks @doberst .. no worries, so in my local env what I do is the following: import os #with the var registered in my environment, I do: #then I call GPT as: response = client.chat.completions.create( print(response.choices[0].message.content) #This model name "test" is the name of the instalation but we call one gpt-35-turbo (in azure they don't use the dot) |
Hi @doberst |
Azure OpenAI is a common enterprise option for a lot of companies integrating LLMs into their environment. It's our only option for genAI, so any integration would be great!
The text was updated successfully, but these errors were encountered: