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

The Pong tutorial: atari-py has been deprecated by Open AI #117

Open
octopusinvitro opened this issue Jan 21, 2022 · 2 comments
Open

The Pong tutorial: atari-py has been deprecated by Open AI #117

octopusinvitro opened this issue Jan 21, 2022 · 2 comments

Comments

@octopusinvitro
Copy link
Contributor

octopusinvitro commented Jan 21, 2022

The Pong tutorial works with the currently pinned versions of gym and atari-py in the requirements.txt file.
However, atari-py (which was the Open AI fork of the ALE), was deprecated in September 2021:
https://github.com/openai/atari-py

The replacement is the original ALE, through the ale-py module:
https://github.com/mgbellemare/Arcade-Learning-Environment

In September 2021, atari-py was finally replaced with ale-py in the gym official repository:
openai/gym#2348

In the gym release notes of October 2021, it specifies the correct way to install the ROMs from now on:

# requirements.txt
-gym==0.18.0
-atari-py==0.2.5
+gym[atari,accept-rom-license]

this will update these libraries while still making the tutorial work with no further changes.

Alternatively, this can be done instead:

# requirements.txt
-gym==0.18.0
-atari-py==0.2.5
+autorom~=0.4.2
+gym[atari]

this will install AutoROM independently and then in the docs the user has to be informed to run:

AutoROM --accept-license

This will also make the tutorial run with no further changes.

Regarding cmake, there is no need to install it anymore, as it was needed for atari-py.
However, ffmpeg is still needed by gym's Monitor wrapper, as it produces videos of the agent’s learning.

I am happy to raise a PR for whichever of the two options if you want to go ahead with the update.
I didn't want to do the PR directly because I don't know if it will be a problem for the numpy tutorials to accept this license.
The commit in the numpy tutorials repo where the versions of gym and atari-py where pinned references a PR in the atari-py repository, which as mentioned before, has been deprecated.

Cheers!

P.S. Regarding this issue, it is now indeed trivial to update the library as all that needs to be done is install gym with the accept-rom-license extra, and everything will work, provided there is no issue for this tutorials to accept the license.

@octopusinvitro octopusinvitro changed the title The Pong tutorial may have to be updated, as atari-py has been deprecated by Open AI The Pong tutorial: atari-py has been deprecated by Open AI Jan 21, 2022
@rossbar
Copy link
Collaborator

rossbar commented Jan 24, 2022

Thanks for the nice summary of the situation and the info about autorom. Indeed this looks like a very straightforward fix provided there are no issues with the rom licenses. I haven't had a change to look at that yet, but I agree that needs to be reviewed before we can add it here.

Linking #87 for visibility

@octopusinvitro
Copy link
Contributor Author

No worries, and thanks for taking a look!

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