-
Notifications
You must be signed in to change notification settings - Fork 619
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
feat(sdk): add support for gymnasium env monitoring, in addition to gym #5008
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #5008 +/- ##
==========================================
+ Coverage 83.52% 83.87% +0.35%
==========================================
Files 295 287 -8
Lines 37366 36683 -683
==========================================
- Hits 31210 30769 -441
+ Misses 6156 5914 -242
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Given that Gym will no longer be maintained and all the major projects have agreed to switch over, you may want to do the opposite (e.g. use Gymnasium and make your monitoring work for Gym if they have it installed instead) for better future proofing |
…ym (#5008) * feat(sdk): add support for gymnasium env monitoring, in addition to gym
Fixes #4985
Description
The
gym
library is not maintained anymore,gymnasium
is the drop-in replacement. This PR makes our current gym monitoring work in case the user hasgymnasium
installed, without changing the current API.Testing
How was this PR tested?
Checklist