Multi Agent DDPG

[veds12]

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

• Complete PettingZooInterface
• test training
• Other misc things

[codecov]

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%)	⬇️
genrl/core/buffers.py	35.29% <12.82%> (-59.95%)	⬇️
genrl/utils/utils.py	18.27% <16.66%> (-77.13%)	⬇️
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%)	⬇️
genrl/agents/deep/dqn/noisy.py	0.00% <0.00%> (-100.00%)	⬇️
genrl/agents/deep/dqn/utils.py	0.00% <0.00%> (-100.00%)	⬇️
genrl/agents/deep/ppo1/ppo1.py	0.00% <0.00%> (-100.00%)	⬇️
genrl/agents/deep/dqn/double.py	0.00% <0.00%> (-100.00%)	⬇️
... and 75 more

[lgtm-com]

This pull request introduces 7 alerts when merging 330adbd into b8a45ab - 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]

This pull request introduces 15 alerts when merging 256f705 into 25eb018 - 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]

This pull request introduces 11 alerts when merging a564e5f into 25eb018 - 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]

This pull request introduces 13 alerts when merging d83a841 into 25eb018 - 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