You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The example code for how to actually use MADDPG, as listed in the documentation, is here: wsjeon/maddpg-rllib#7
Based on the issues there, the code example couldn't have even been tested before submission, PR key fixes for basic functionality aren't being merged/looked into. I understand that maddpg is still /contrib, but this is a real problem, and needs to be moved to someones library who will merge PRs, etc. I'm happy to fork @wsjeon 's code, fix the instructions, set up an accurate requirements.txt, include the PRs, etc. if you guys would want to change the official pointer to my fork. (I've been doing a ton of work with the MADDPG library recently).
Thanks for looking at this @justinkterry . I think we can make a doc update to point to the new maintained repo as an update over the original -- feel free to submit a PR for this once is ready.
The example code for how to actually use MADDPG, as listed in the documentation, is here: wsjeon/maddpg-rllib#7
Based on the issues there, the code example couldn't have even been tested before submission, PR key fixes for basic functionality aren't being merged/looked into. I understand that maddpg is still /contrib, but this is a real problem, and needs to be moved to someones library who will merge PRs, etc. I'm happy to fork @wsjeon 's code, fix the instructions, set up an accurate requirements.txt, include the PRs, etc. if you guys would want to change the official pointer to my fork. (I've been doing a ton of work with the MADDPG library recently).
cc @ericl
The text was updated successfully, but these errors were encountered: