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

Add OpenAI models from Jan 25th #150

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

philippzagar
Copy link
Contributor

@philippzagar philippzagar commented Jan 28, 2024

What

Update the OpenAI library with the newest released OpenAI models from January 25th.
Most importantly, the PR introduces support for the gpt-4-turbo-preview (alias for gpt-4-0125-preview) and gpt-3.5-turbo-0125 (aliased by gpt-3.5-turbo) models.

Why

Developers are now able to use the newest models within their applications.

Affected Areas

Only minor adjustments within the Model.swift file, as well as the README.

@philippzagar
Copy link
Contributor Author

@Krivoblotsky @ingvarus-bc @SunburstEnzo Would be great to get this out quickly so that people can start using the new models, the PR contains only minor code changes 🚀

README.md Outdated Show resolved Hide resolved
@ingvarus-bc
Copy link
Contributor

ingvarus-bc commented Feb 2, 2024

Hi, @philippzagar, thanks for your PR!
It seems like moderations model has been changed and now tests for it are failing. Perhaps changes to moderations from this PR could fix it? 🤔

UPD: Found the issue. Just little update to test data is required 😁

Copy link

sonarcloud bot commented Feb 2, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

5 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@philippzagar
Copy link
Contributor Author

@ingvarus-bc @SunburstEnzo Please approve the workflow execution once again and then we can merge the PR 🚀

@ingvarus-bc ingvarus-bc merged commit 27ad53b into MacPaw:main Feb 2, 2024
4 checks passed
@philippzagar
Copy link
Contributor Author

@ingvarus-bc Would you mind tagging a new release?

@philippzagar
Copy link
Contributor Author

@ingvarus-bc @SunburstEnzo As tons of new changes have been merged, would you mind tagging a new release for the library?

@jagilley
Copy link

Yes please, +1 on tagging a new release @ingvarus-bc

@philippzagar
Copy link
Contributor Author

@ingvarus-bc @SunburstEnzo @jagilley Created a new issue to track the new release: #171

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.

4 participants