-
Notifications
You must be signed in to change notification settings - Fork 309
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
Resolve conflicts between mujoco-py and gym temporarily #2188
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2188 +/- ##
==========================================
- Coverage 91.36% 91.17% -0.20%
==========================================
Files 198 198
Lines 10944 10912 -32
Branches 1374 1367 -7
==========================================
- Hits 9999 9949 -50
- Misses 688 703 +15
- Partials 257 260 +3
Continue to review full report at Codecov.
|
Unfortunately, the only way to solve this is to fix the build under the new pip -- we have no control over which version of pip people use, and when in doubt we should use the latest. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment to setup.py explaining what's going on here? As is this change is pretty unclear.
Sure |
* Pin pip to 20.2 * Pin pip to 20.2.4 * Pin pip to 20.2.4 * Resolve conflict between mujoco-py and gym * Add comments
Pip release 20.3 today, which makes the CI build failed. Here is the log, and the related issue.