NVlabs / progprompt-vh

ProgPrompt for Virtualhome
Other
106 stars 12 forks source link

OpenAI API Update #3

Open Panjete opened 9 months ago

Panjete commented 9 months ago

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

sleep-at-9 commented 5 months ago

Dear friend,I follow your suggestions,upgrade openai to 1.14.3 and use "openai.completions.create" instead of "open.Completion.create", the location of virtualhome is here image My operating system is win10.I already run the VirtualHome.exe when execute this command with arguments "--progprompt-path E:\experiment\demo\progprompt-vh-main\scripts --expt-name exp01 --openai-api-key sk-Hjr****C3 --display 0 --gpt-version gpt-3.5-turbo-instruct"

Such an error has occurred:I guess This error occurs because of a communication problem with the emulator.

error

and what does the seed"f84ee28a75b23318ee1bf652862b1c993269cd06" mean?Do I need to set it up somewhere? image

thank you,my friend!

Panjete commented 5 months ago

@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 .

sleep-at-9 commented 5 months ago

Thank you for your reply! I have tested VirtualHome and ran the Demo of VirtualHome successfully.I will test my mistakes from a different angle. Your reply has given me the motivation to continue my research. Best wish! Long may the sunshine!