-
Notifications
You must be signed in to change notification settings - Fork 106
Support Windows Platform #227
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you check that this worked on unix-based environments as well? I will try and confirm that it does before merging this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I haven't check it on unix-based environments. It will be great if you can do this.
|
@NingLi670 @crawles Tried it on windows. I am running cmd in admin mode. |
|
@anunay1 Thanks for your feedback! But sorry I can't pinpoint the exact problem. Can you provide more information? You can change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested it on UNIX (Mac).
|
Thank for sending this :D. Will follow-up with additional fixes |
|
@NingLi670 @crawles seems like it's not able to tun adb commands I0306 12:03:41.837401 19836 emulator_simulator.py:110] Connecting to existing emulator "'emulator-5554'" |
|
Followed the above. still throws error: I0306 13:51:02.918175 12680 emulator_simulator.py:110] Connecting to existing emulator "'emulator-5554'" |
|
@anunay1 If it works whell, you can use |
|
My bad, messed up with adb path. Thanks its working now. |
|
How to solve the "source" part of the source code is a Linux module instead of a windows module, how to modify this part, thank you! |
|
Can you share more of the error please? |
|
@Charles-Donne See PR #247 |
|
I would like to ask how to set the url and key of the large model in the window environment, thank you |
It's settled, thank you. But I want to ask why it fails to call the api. Is there any requirement |
Are there any restrictions on which large models can be called? response = requests.post( Can I use it? How do you use the large language model api? I don't know what the problem is, please help answer, thank you! |
Support Windows Platform
Support Windows Platform
Three changes make AndroidWorld runnable on Windows operating system:
/tmpto automatically obtain it according to the operating system.os.path.joinintostr(Path(*args).as_posix())to adapt to the path in posix format.strip()to deal with\rin Windows platform.More changes you need to do to make it work on WIndows Platform:
~/Library/Android/sdk/emulator/emulator, start theemulator.exefrom another path. Usually it is~\AppData\Local\Android\Sdk\emulator\emulator.exe._find_adb_directoryinminimal_task_runner.pyandrun.py._increase_file_descriptor_limitinenv_launcher.py(Fixed in PR Fix resource package import error in Windows platform #247).