Skip to content
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

What version of MAgent is compatible with MARLlib? #81

Closed
Morphlng opened this issue Feb 19, 2023 · 2 comments
Closed

What version of MAgent is compatible with MARLlib? #81

Morphlng opened this issue Feb 19, 2023 · 2 comments

Comments

@Morphlng
Copy link

According to the doc about Environments, it would be fine to just use pip install pettingzoo[magent]. However, the latest version of pettingzoo has moved MAgent to a dedicated project, so its unusable now.

I've compared the code in envs/base_env/magent.py with pettingzoo's previous version.

from pettingzoo.magent import adversarial_pursuit_v3, battle_v3, battlefield_v3, combined_arms_v5, gather_v3, \
tiger_deer_v3
REGISTRY = {}
REGISTRY["adversarial_pursuit"] = adversarial_pursuit_v3.parallel_env
REGISTRY["battle"] = battle_v3.parallel_env
REGISTRY["battlefield"] = battlefield_v3.parallel_env
REGISTRY["combined_arms"] = combined_arms_v5.parallel_env
REGISTRY["gather"] = gather_v3.parallel_env
REGISTRY["tiger_deer"] = tiger_deer_v3.parallel_env

It seems like we have to use version before 1.15.0 (e.g. 1.14.0). Yet, after pip install pettingzoo[magent]==1.14.0 and run a test, it reports problem as: "cannot import name 'aec_to_parallel' from 'pettingzoo.utils.conversions'"

I've tried newer version of MAgent and changed magent.py accordingly, but it will raise other problems.

So which version has been tested exactly?

@Morphlng
Copy link
Author

Okay I've found a solution. The following version works for me pip install supersuit==3.3.2 pettingzoo[magent]==1.14.0. Higher version of pettingzoo doesn't match the supersuit check.

I think the document/readme need an update.

@Theohhhu
Copy link
Collaborator

A requirements.txt is here for you to refer. Yet there is no need to strictly follow it.

The document/readme will be updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants