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

updated openai to 1.1.1 #453

Merged
merged 3 commits into from
Nov 18, 2023
Merged

updated openai to 1.1.1 #453

merged 3 commits into from
Nov 18, 2023

Conversation

gilcu3
Copy link
Contributor

@gilcu3 gilcu3 commented Nov 8, 2023

This PR adds the minimal necessary changes to support the new version of the openai python library, specifically the current version 1.1.1. I tested all the main functionality (private chats, group chats, plugins, images, transcription, stream) and it seems to work. I had my doubts on some functions in the utils.py file, please check those are used as expected before merging. Most changes done were related to changes on naming and the use of the new client object, and followed the migration guide here.

Having this update I hope we will be able to use the new vision and TTS models soon :)

@gilcu3
Copy link
Contributor Author

gilcu3 commented Nov 8, 2023

Once this is merged, I can open another PR with initial support for tts done in fork

@AlexHTW
Copy link
Contributor

AlexHTW commented Nov 8, 2023

Once this is merged, I can open another PR with initial support for tts done in fork

That's awesome!
How do you feel about already submitting a PR marked as WIP so that you can get feedback already and everybody can see that this feature is in the pipeline?

@gilcu3
Copy link
Contributor Author

gilcu3 commented Nov 8, 2023

That's awesome! How do you feel about already submitting a PR marked as WIP so that you can get feedback already and everybody can see that this feature is in the pipeline?

As that branch depends on this one, I thought it would be better to wait a bit. But yeah, WIP PR sounds fine as well.

This was referenced Nov 8, 2023
@n3d1117
Copy link
Owner

n3d1117 commented Nov 18, 2023

Thanks @gilcu3, I've bumped openai-python from 1.1.1 to 1.3.3 directly since there are no major changes involved

@n3d1117 n3d1117 merged commit 2a0d1aa into n3d1117:main Nov 18, 2023
@gilcu3 gilcu3 deleted the openai-v1.1.1 branch November 18, 2023 15:49
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.

None yet

3 participants