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/fix openai #882

Closed
wants to merge 2 commits into from
Closed

Fix/fix openai #882

wants to merge 2 commits into from

Conversation

stvgz
Copy link

@stvgz stvgz commented Dec 3, 2023

UI Shows old way of call openai isn't supprted anymore.
(Screen shot missing)
Based on new guidlines from openai

  1. Create instance of client OpenAI
  2. Do not rely on global key but pass the openai_key directly to the client

UI after change
image


Not good code. Just working. Please reject if better solution provided. Thanks

@csunny
Copy link
Collaborator

csunny commented Dec 4, 2023

@stvgz Hi, some conflicts need to be resolved, conflicting file pilot/model/proxy/llms/chatgpt.py.
by the way, you need to consider the compatibility of old versions of OpenAI API.

@stvgz stvgz closed this by deleting the head repository Dec 5, 2023
@stvgz
Copy link
Author

stvgz commented Dec 5, 2023

Looks like it's already fixed 3 weeks ago.
e198cd3

PR closed

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