Skip to content

Conversation

chriselion
Copy link
Contributor

Proposed change(s)

Update all pip install instructions for mlagents and mlagents_envs to specify the released pypi version. This standardizes on the command to be "python -m pip" instead of "pip" or "pip3", which is the recommended approach (see link).

This will run at the same time that we update the release_xyz_docs links.

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

https://www.youtube.com/watch?list=PLWBKAf81pmOaP9naRiNAqug6EBnkPakvY&v=gnYdk_U6UQ0&feature=youtu.be

@chriselion chriselion requested review from surfnerd and ervteng March 10, 2021 20:21
or PIP_INSTALL_PATTERN.search(line) is None
or pip_allow_pattern.search(line) is not None
)
# if not pip_install_ok:
Copy link
Contributor

Choose a reason for hiding this comment

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

should this comment be here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, that was debugging, will remove.

@chriselion chriselion merged commit d18d5ff into main Mar 10, 2021
@delete-merged-branch delete-merged-branch bot deleted the MLA-1731-update-pip-instructions branch March 10, 2021 23:39
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 11, 2022
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.

2 participants