-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Adjust the azure model deployment name call corresponding to README #309
Conversation
Codecov Report
@@ Coverage Diff @@
## master #309 +/- ##
=======================================
Coverage 92.63% 92.63%
=======================================
Files 22 22
Lines 679 679
=======================================
Hits 629 629
Misses 37 37
Partials 13 13 |
Thank you for the PR! |
README.md
Outdated
config := openai.DefaultAzureConfig("your Azure OpenAI Key", "https://your Azure OpenAI Endpoint ", "your Model deployment name") | ||
config := openai.DefaultAzureConfig("your Azure OpenAI Key", "https://your Azure OpenAI Endpoint") | ||
config.AzureModelMapperFunc = func(model string) string { | ||
azureModelMapping = []string{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR! Could you please comment out the config.AzureModelMapperFunc
section and clearly indicate that it is optional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course, great idea
I've try it in my application and it should be:
|
Modify variable type definition error
This problem has been fixed, thanks |
According to the modification of #306, adjust the azure model deployment name call corresponding to README