Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Deprecate old atari_wrappers #104

Merged
merged 3 commits into from
Nov 17, 2022
Merged

Deprecate old atari_wrappers #104

merged 3 commits into from
Nov 17, 2022

Conversation

xiaomengy
Copy link
Contributor

@xiaomengy xiaomengy commented Nov 17, 2022 •
edited
Loading

This PR deprecate the old Atari Wrappers.

  1. Switch to Atari-v5 env as suggested in https://brosa.ca/blog/ale-release-v0.7
  2. Use gym.wrappers.AtariPreprocessing to replace old atari_wrappers.
  3. Add random seed for model server.
  4. Remove TimeLimitWrapper and switch to gym.wrappers.TimeLimit.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 17, 2022
@xiaomengy xiaomengy merged commit 55775d0 into facebookresearch:main Nov 17, 2022
@xiaomengy xiaomengy deleted the new_gym_api branch November 17, 2022 08:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@xiaomengy @facebook-github-bot