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

Updated gym version requirement to 0.21.* #5

Open
wants to merge 1 commit into
base: pyglet1.5
Choose a base branch
from

Conversation

Axle7XStriker
Copy link

This update is required by https://github.com/AlignmentResearch/rllf to depend on magical and create custom environments for their research. However, it doesn't make any significant updates but rather updates the gym version requirement for magical which seems to pass the current test suite.

@qxcv
Copy link
Owner

qxcv commented May 30, 2023

Thanks for the PR. Unfortunately when I try to install I get this error:

$ pip install -e .
Obtaining file:///home/sam/repos/magical-pr
  Preparing metadata (setup.py) ... done
Collecting pymunk~=5.6.0
  Using cached pymunk-5.6.0-py2.py3-none-manylinux1_x86_64.whl (535 kB)
Collecting pyglet==1.5.*
  Using cached pyglet-1.5.27-py3-none-any.whl (1.1 MB)
Collecting gym==0.21.*
  Using cached gym-0.21.0.tar.gz (1.5 MB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error
  
  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [1 lines of output]
      error in gym setup command: 'extras_require' must be a dictionary whose values are strings or lists of strings containing valid project/version requirement specifiers.
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

I have:

$ python --version
Python 3.8.16
$ pip --version
pip 23.0.1 from /home/sam/.../pip (python 3.8)

Any idea what might be causing this?

@Axle7XStriker
Copy link
Author

Try downgrading setuptools version to <=v65.5.0. The later versions throw this error.

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

Successfully merging this pull request may close these issues.

2 participants