-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Question] Why gymnasium is pinned to 0.28.1 ? #1452
Comments
Hello, TL;DR: The last year and a half has been a real pain in the neck for the SB3 devs, each new gym/gymnasium release came with breaking changes (more or less documented), so until gym is actually stable again, we have to pin to prevent any nasty surprises. Note: it doesn't prevent you from installing ray by using EDIT: I would rather push rllib to support latest Gymnasium version. |
Yes I know that, that is why I suggest a range instead of a single version. It already pervents this kind of issues. If you tell me this is only for typing, this is not a concern for 99% of the users, so it is a bit sad to make the life harder for the all community because of this. Does the range I suggest would be suitable except for typing ? If so, it may be a good compromise no ? If the user is doing a clean install the latest gymnasium version being compatible will be installed so it sounds good !
Yes and no. It will not work in general because you actaully need to install deps most of the time.
Ideally, but it is a company-owned project, so it is much harder to make it move forward. Plus their next release is not expected before 2 months at least if it goes as usual. |
So, unfortunately, in that case, SB3 breaks with earlier version of Gymnasium (Farama-Foundation/Gymnasium#337, ad48559):
company-owned project should also mean more resources... |
If stable-baseline3 is failing at runtime for any other version of
Yes and no. They do, but only for what really matters for them and their clients, never for the open-source community. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Closing as original question was answered and PR is on its way in RLLib repo: ray-project/ray#35698 |
❓ Question
Is enforcing
gymnasium==0.28.1
is strict requirement ?ray[rllib]
has also pinnedgymnasium
but using a different version (0.26.3
), so that installing bothstable-baseline3
andray[rllib]
at the same time is raising an exception. Would be be possible to relax this requirement, typically by allowing the range of versionsgynasium >= 0.26.3, <= 0.28.1
?Checklist
The text was updated successfully, but these errors were encountered: