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

GitHub Repo Q&A: Fixed issue that prevents choosing and saving an OpenAI completions model. #1305

Merged

Conversation

alliscode
Copy link
Member

Motivation and Context

When using OpenAI in the GitHub Repo Q&A sample, an error occurs when a valid chat completions model is chosen from the dropdown list. Furthermore, selecting a text completions model passes validation but causes a bad request error when saving.

Description

Fixed issue by updating the model validation function to test model selection against the chat completions endpoint rather than the text completions endpoint.

Contribution Checklist

… the chat completions endpoint rather than the text completions endpoint.
@adrianwyatt adrianwyatt self-requested a review June 1, 2023 19:05
@adrianwyatt adrianwyatt self-assigned this Jun 1, 2023
@adrianwyatt adrianwyatt added the PR: ready for review All feedback addressed, ready for reviews label Jun 1, 2023
@adrianwyatt adrianwyatt added PR: ready to merge PR has been approved by all reviewers, and is ready to merge. and removed PR: ready for review All feedback addressed, ready for reviews labels Jun 1, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) June 1, 2023 19:07
@adrianwyatt adrianwyatt merged commit 1a49468 into microsoft:main Jun 1, 2023
salmon131 pushed a commit to salmon131/semantic-kernel that referenced this pull request Jun 3, 2023
…nAI completions model. (microsoft#1305)

### Motivation and Context
When using OpenAI in the GitHub Repo Q&A sample, an error occurs when a
valid chat completions model is chosen from the dropdown list.
Furthermore, selecting a text completions model passes validation but
causes a bad request error when saving.

### Description
Fixed issue by updating the model validation function to test model
selection against the chat completions endpoint rather than the text
completions endpoint.
shawncal pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
…nAI completions model. (microsoft#1305)

### Motivation and Context
When using OpenAI in the GitHub Repo Q&A sample, an error occurs when a
valid chat completions model is chosen from the dropdown list.
Furthermore, selecting a text completions model passes validation but
causes a bad request error when saving.

### Description
Fixed issue by updating the model validation function to test model
selection against the chat completions endpoint rather than the text
completions endpoint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants