3rd-party env Integration

[KornbergFresnel]

Enable pettingzoo again, not fully test yet.

• [x] SMAC
• [x] COMA
• [x] TeamAgent

[codecov[bot]]

Codecov Report

Merging #54 (38cd115) into main (f3fa6dc) will increase coverage by 2.02%. The diff coverage is 85.41%.

:exclamation: Current head 38cd115 differs from pull request most recent head abadacf. Consider uploading reports for the commit abadacf to get more accurate results

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   82.19%   84.21%   +2.02%     
==========================================
  Files          36       38       +2     
  Lines        2482     2636     +154     
==========================================
+ Hits         2040     2220     +180     
+ Misses        442      416      -26     

Impacted Files	Coverage Δ
malib/agent/indepdent_agent.py	66.66% <0.00%> (-8.34%)	:arrow_down:
malib/rl/dqn/policy.py	70.21% <0.00%> (ø)
malib/backend/offline_dataset_server.py	86.11% <42.85%> (-3.89%)	:arrow_down:
malib/agent/agent_interface.py	70.81% <61.53%> (-1.66%)	:arrow_down:
malib/rl/common/policy.py	65.97% <66.66%> (-2.94%)	:arrow_down:
malib/rollout/envs/pettingzoo/env.py	67.64% <67.64%> (ø)
malib/rl/coma/trainer.py	96.06% <96.06%> (ø)
malib/rl/a2c/trainer.py	90.32% <100.00%> (+0.15%)	:arrow_up:
malib/rl/coma/critic.py	100.00% <100.00%> (ø)
malib/rl/common/trainer.py	94.33% <100.00%> (+1.88%)	:arrow_up:
... and 13 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[lgtm-com[bot]]

This pull request introduces 1 alert and fixes 2 when merging ef4a5f4218e1927219aaf208d7f11b5d4789d0ef into f3fa6dc0abf21acbce1eb2d825ab0c8eb6f7bb29 - view on LGTM.com

new alerts:

• 1 for Missing call to `__init__` during object initialization

fixed alerts:

• 2 for Missing call to `__init__` during object initialization

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 8 when merging 714c8e61ef4167655c8202e17e23bc5ba33b513e into b8a1adc9c6db4afc6f62c5f20cdd180b5af5009a - view on LGTM.com

new alerts:

• 3 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for Module is imported with 'import' and 'import from'

fixed alerts:

• 2 for Unused local variable
• 2 for Unused import
• 2 for Missing call to `__init__` during object initialization
• 1 for Wrong name for an argument in a call

• 1 for Module is imported with 'import' and 'import from'

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com[bot]]

This pull request introduces 12 alerts and fixes 9 when merging b0eea658ba2efcd8bc579440471e348a6159cde2 into 0088f2adb0f1007a1711b0df55799a6e1ace440c - view on LGTM.com

new alerts:

• 3 for Unused local variable
• 3 for Unused import
• 3 for Module is imported with 'import' and 'import from'
• 2 for Wrong number of arguments in a call
• 1 for Missing call to `__init__` during object initialization

fixed alerts:

• 3 for Unused import
• 2 for Unused local variable
• 2 for Missing call to `__init__` during object initialization
• 1 for Wrong name for an argument in a call

• 1 for Module is imported with 'import' and 'import from'

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com[bot]]

This pull request introduces 12 alerts and fixes 9 when merging 6fa44bb54a9839e0c8ea7ab5f4cf27511cdfe2d1 into 0088f2adb0f1007a1711b0df55799a6e1ace440c - view on LGTM.com

new alerts:

• 3 for Unused local variable
• 3 for Unused import
• 3 for Module is imported with 'import' and 'import from'
• 2 for Wrong number of arguments in a call
• 1 for Missing call to `__init__` during object initialization

fixed alerts:

• 3 for Unused import
• 2 for Unused local variable
• 2 for Missing call to `__init__` during object initialization
• 1 for Wrong name for an argument in a call

• 1 for Module is imported with 'import' and 'import from'

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com[bot]]

This pull request introduces 13 alerts and fixes 9 when merging 38cd115aabbb10cb1cbce880a5cde0a814d3ca72 into 0088f2adb0f1007a1711b0df55799a6e1ace440c - view on LGTM.com

new alerts:

• 5 for Unused import
• 3 for Unused local variable
• 3 for Module is imported with 'import' and 'import from'
• 1 for Missing call to `__init__` during object initialization
• 1 for Wrong number of arguments in a call

fixed alerts:

• 3 for Unused import
• 2 for Unused local variable
• 2 for Missing call to `__init__` during object initialization
• 1 for Wrong name for an argument in a call

• 1 for Module is imported with 'import' and 'import from'

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com[bot]]

This pull request introduces 11 alerts and fixes 9 when merging f66ba95485f583b0962dd2200a672eff380d6d3b into 0088f2adb0f1007a1711b0df55799a6e1ace440c - view on LGTM.com

new alerts:

• 6 for Unused import
• 3 for Module is imported with 'import' and 'import from'
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization

fixed alerts:

• 3 for Unused import
• 2 for Unused local variable
• 2 for Missing call to `__init__` during object initialization
• 1 for Wrong name for an argument in a call

• 1 for Module is imported with 'import' and 'import from'

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.