Multi Agent DDPG

[veds12]

trafficstars

This is a very barebones implementation of maddpg. To Do :

• [x] Complete PettingZooInterface
• [ ] test training
• [ ] Other misc things

[codecov[bot]]

Codecov Report

Merging #377 into master will decrease coverage by 68.14%. The diff coverage is 12.90%.

@@             Coverage Diff             @@
##           master     #377       +/-   ##
===========================================
- Coverage   91.28%   23.13%   -68.15%     
===========================================
  Files          90       90               
  Lines        3809     3933      +124     
===========================================
- Hits         3477      910     -2567     
- Misses        332     3023     +2691     

Impacted Files	Coverage Δ
genrl/core/rollout_storage.py	25.82% <9.21%> (-63.80%)	:arrow_down:
genrl/core/buffers.py	35.29% <12.82%> (-59.95%)	:arrow_down:
genrl/utils/utils.py	18.27% <16.66%> (-77.13%)	:arrow_down:
genrl/core/__init__.py	100.00% <100.00%> (ø)
genrl/utils/__init__.py	100.00% <100.00%> (ø)
genrl/trainers/__init__.py	0.00% <0.00%> (-100.00%)	:arrow_down:
genrl/agents/deep/dqn/noisy.py	0.00% <0.00%> (-100.00%)	:arrow_down:
genrl/agents/deep/dqn/utils.py	0.00% <0.00%> (-100.00%)	:arrow_down:
genrl/agents/deep/ppo1/ppo1.py	0.00% <0.00%> (-100.00%)	:arrow_down:
genrl/agents/deep/dqn/double.py	0.00% <0.00%> (-100.00%)	:arrow_down:
... and 75 more

[lgtm-com[bot]]

This pull request introduces 7 alerts when merging 330adbd853641a5fdeae7333d7e8761b28e1edd0 into b8a45ab7fd058d120acd058ddead1db77c9bb616 - view on LGTM.com

new alerts:

• 5 for Unused import
• 1 for First parameter of a method is not named 'self'
• 1 for Syntax error

[lgtm-com[bot]]

This pull request introduces 15 alerts when merging 256f705c568fb7ec1f61e35b04c8d5d83e61b3d3 into 25eb018f18a9a1d0865c16e5233a2a7ccddbfd78 - view on LGTM.com

new alerts:

• 7 for Unused import
• 2 for Unused local variable
• 2 for Suspicious unused loop iteration variable
• 2 for Variable defined multiple times
• 1 for First parameter of a method is not named 'self'
• 1 for Wrong number of arguments in a call

[lgtm-com[bot]]

This pull request introduces 11 alerts when merging a564e5f1e059f3920420936b34ed1ffbe9d18c55 into 25eb018f18a9a1d0865c16e5233a2a7ccddbfd78 - view on LGTM.com

new alerts:

• 6 for Unused import
• 2 for Suspicious unused loop iteration variable
• 2 for Variable defined multiple times
• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 13 alerts when merging d83a841418ab6d45ec10a0663d84ea50c9b8d32f into 25eb018f18a9a1d0865c16e5233a2a7ccddbfd78 - view on LGTM.com

new alerts:

• 8 for Unused import
• 2 for Suspicious unused loop iteration variable
• 1 for Unused local variable
• 1 for Variable defined multiple times
• 1 for Redundant assignment