-
Notifications
You must be signed in to change notification settings - Fork 152
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
Glenn unification multiagent support #375
Conversation
Gets rid of shutil.Error by not copying the cache.properties.lock
Multiple environment test
Removing unneeded Render command Moved _find_server back to where it can still influence the xml. Correctly tick client/server and process observations. Branch using correct client-tick obs. Fixed log once functionality. Fixed the env stepper Fixed issues syncing two clients and stepping envs. Remove dead code. Moved the multiagent test xmls and script to tests dir. Fixed action handlers. obs/ac are now dicts to match the interface of magym.
Hello @glennpow! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2020-08-18 21:12:23 UTC |
There is currently an issue processing the info-dict, that I didn't have time to resolve, but otherwise it seems very close to working. You can run this from the root dir to test
python tests/multiagent_test.py
. When working, it should create the env with 2 agents and run an episode.