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

Upgrade examples to latest version (no legacy models) #697

Merged
merged 8 commits into from
Dec 18, 2023

Conversation

franz101
Copy link
Contributor

@franz101 franz101 commented Nov 7, 2023

text-davinci is deprecated therefore using gpt-3.5-turbo-instruct

@franz101 franz101 requested a review from a team as a code owner November 7, 2023 00:05
@franz101
Copy link
Contributor Author

franz101 commented Nov 7, 2023

@yutian-openai

@rattrayalex
Copy link
Collaborator

Thank you for this; we actually should update the examples to show chat completions instead of completions. If you'd like to update the examples accordingly, it'd be much appreciated! If not, we can take care of that soon.

@rattrayalex
Copy link
Collaborator

cc @logankilpatrick

@franz101
Copy link
Contributor Author

@logankilpatrick logankilpatrick merged commit 7367256 into openai:main Dec 18, 2023
1 check passed
megamanics pushed a commit to devops-testbed/openai-python that referenced this pull request Aug 14, 2024
* Update streaming.py

Removed legacy model

* Update streaming.py

* more upgrades
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
* Update streaming.py

Removed legacy model

* Update streaming.py

* more upgrades
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.

3 participants