semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard

OpenAI config sample no need to pass the service name

Open michaelslz23 opened this issue 2 years ago • 1 comments

Describe the bug https://github.com/microsoft/semantic-kernel/blob/main/dotnet/README.md sample codes for OpenAI // Alternative using OpenAI // kernel.Config.AddOpenAITextCompletionService("davinci-openai", // "text-davinci-003", // OpenAI Model name // "...your OpenAI API Key..." // OpenAI API Key // );

"davinci-openai" is not needed here. Otherwise you'll get 401 permission error.

michaelslz23 avatar May 12 '23 18:05 michaelslz23

Good catch!

Would you be interested in sending a PR that fixes this? If not, no worries, we can take care of it.

craigomatic avatar May 16 '23 21:05 craigomatic

@craigomatic Thanks. I'm not sure if I did it right or not, pull request is here, https://github.com/microsoft/semantic-kernel/pull/1078.

michaelslz23 avatar May 18 '23 21:05 michaelslz23

Looks like there were overlapping PRs to fix this one and another member of the community was slightly faster at clicking the button to resolve.

Thank you in any case, closing this for now :)

craigomatic avatar May 19 '23 00:05 craigomatic