Skip to content

pin pip to 20.2 for now #4694

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

Merged
merged 2 commits into from
Nov 30, 2020
Merged

pin pip to 20.2 for now #4694

merged 2 commits into from
Nov 30, 2020

Conversation

chriselion
Copy link
Contributor

@chriselion chriselion commented Nov 30, 2020

Proposed change(s)

pip 20.3 throws an exception trying to install our dependencies. pin to 20.2 until they get a fix

We might need this later for tests that run on yamato, but as of the moment, the pypi mirror we use internally hasn't gotten 20.3

Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)

pypa/pip#9180

Types of change(s)

  • Bug workaround

@chriselion chriselion requested a review from andrewcoh November 30, 2020 19:22
@@ -8,6 +8,7 @@ on:
- 'gym-unity/**'
- 'test_constraints*.txt'
- 'test_requirements.txt'
- '.github/workflows/pytest.yml'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure tests rerun if this file changes.

@@ -44,7 +45,8 @@ jobs:
run: python -c "import sys; print(sys.version)"
- name: Install dependencies
run: |
python -m pip install --upgrade pip
# pin pip to workaround https://github.com/pypa/pip/issues/9180
python -m pip install pip==20.2

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
python -m pip install pip==20.2
python -m pip install pip==20.2.4

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion, but I think this is enough for now. Hopefully you'll have a patch release soon :)

@chriselion chriselion merged commit 3c07161 into master Nov 30, 2020
@delete-merged-branch delete-merged-branch bot deleted the pip-workaround branch November 30, 2020 20:55
@chriselion chriselion mentioned this pull request Feb 12, 2021
1 task
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants