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

fix: Replace deprecated text-davinci-003 model with gpt-3.5-turbo-instruct model #6660

Merged
merged 3 commits into from
Dec 29, 2023

Conversation

julian-risch
Copy link
Member

@julian-risch julian-risch commented Dec 28, 2023

Related Issues

Proposed Changes:

Replace all usage of the deprecated text-davinci-003 model with the gpt-3.5-turbo-instruct model

How did you test it?

CI. I expect some tests to fail because of slightly different LLM outputs.

Notes for the reviewer

I opened a PR for updating the tutorials too. No updates are needed for 2.0.

Checklist

@julian-risch julian-risch requested review from vblagoje and removed request for anakin87 December 28, 2023 17:25
Copy link
Member

@vblagoje vblagoje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Played with examples locally. Works well. LGTM.

@julian-risch julian-risch merged commit ca812cb into v1.x Dec 29, 2023
63 checks passed
@julian-risch julian-risch deleted the replace-davinci-model branch December 29, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:pipeline topic:promptnode topic:tests type:documentation Improvements on the docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants