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

#42 Deprecation of TextDavinci003Engine Model #43

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pradytpk
Copy link

@pradytpk pradytpk commented Feb 20, 2024

#42
For the text completions of openai the following models are deprecated as of 2024-01-04

TextAda001Engine     = "text-ada-001"
TextBabbage001Engine = "text-babbage-001"
TextCurie001Engine   = "text-curie-001"
TextDavinci001Engine = "text-davinci-001"
TextDavinci002Engine = "text-davinci-002"
TextDavinci003Engine = "text-davinci-003"
AdaEngine            = "ada"
BabbageEngine        = "babbage"
CurieEngine          = "curie"
DavinciEngine        = "davinci"

Reference
https://platform.openai.com/docs/deprecations/instructgpt-models
https://platform.openai.com/docs/deprecations/fine-tuning-gpt-models

Currently supported model is

"gpt-3.5-turbo-instruct"
"babbage-002"
"davinci-002"

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

✅ This pull request was sent to the PullRequest network because it was selected by the Smart Review mechanism.

Smart Review Evaluation

  79.9%  # original selection chance (based on the amount of review performed so far vs predicted)
* 1.30   # large_diff modifier
* 0.68   # security_risk modifier
-------
= 71.2%  # updated selection chance
> 0.159  # roll (must be lower than the selection chance to be selected)

@pradytpk you can click here to see the review status or cancel the code review job.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

PullRequest Breakdown

Reviewable lines of change

+ 4
- 11

100% Go

Type of change

Minor Update - These changes appear to be a minor update to existing functionality and features.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

Overall the changes look good. I just had one minor question on line 36 about whether some of the other engine types (lines 16-27) should also be updated or deprecated as well since the reference indicates some other models are deprecated. But other than that question, the changes seem appropriate based on the provided deprecation reference. Nice work!

Image of Xiaoyong W Xiaoyong W


Reviewed with ❤️ by PullRequest

gpt3.go Show resolved Hide resolved
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

PullRequest reviewed the updates made to #43 up until the latest commit (eb63db0). No further issues were found.

Reviewed by:

Image of Xiaoyong W Xiaoyong W

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

Due to inactivity, PullRequest has cancelled this review job. You can reactivate the code review job from the PullRequest dashboard.

@vishalvivekm
Copy link

I also get same error

[404:invalid_request_error] The model `davinci` has been deprecated, learn more here: https://platform.openai.com/docs/deprecations

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.

2 participants