-
Notifications
You must be signed in to change notification settings - Fork 13
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
OpenAI API Update #3
Comments
@sleep-at-9 I ran code on a linux machine, so there could be some differences to our setups. As a possible debugging step, you should first confirm whether VirtualHome is installed properly and can run (independently of ProgPrompt's code). You can find our implementation and edits here . |
Thank you for your reply! |
@sleep-at-9 Hello! I have met the same problem recently. Do you have any idea how to fix it? Thanks a lot.
|
Kindly Update the code in utils_execute.py to be compatible with the new openai SKD.
When crafting a response (line 38), one now needs to use "openai.completions.create" instead of "open.Completion.create". You can find the link for this conversation : here
I also suggest updating the Readme to also mention the filepath changes that need to be made in virtualhome/virtualhome/init.py , and the location of where virtualhome needs to be cloned w.r.t. progprompt, to make setup easier for the first-time user.
Sincerely
The text was updated successfully, but these errors were encountered: