Skip to content

Conversation

Tessil
Copy link
Contributor

@Tessil Tessil commented May 21, 2020

Hi,

This PR adds a note in the TensorFlow Model Optimization installation guide to recommend the usage of Python virtual environments in a way similar to the recommendation in the TensorFlow installation guide.

@googlebot googlebot added cla: yes PR contributor has signed CLA labels May 21, 2020

### Stable Builds

It is recommended to create a Python virtual environment before proceeding to

Choose a reason for hiding this comment

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

Could you move this above the ### Stable Builds header?

This comment is applicable to both Stable Builds and Installing from Source?

Afterwards, squash the commits and we can merge the PR. Thanks for the PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Effectively it's applicable to both. I made the modification, thanks.

@Tessil Tessil force-pushed the toupstream/mot-install-add-virtual-env-note branch from 60ef9d0 to 8ade345 Compare May 21, 2020 22:04
@alanchiao alanchiao added the ready to pull Working to get PR submitted to internal repository, after which merging to Github happens. label May 21, 2020
@copybara-service copybara-service bot merged commit c2642e5 into tensorflow:master May 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes PR contributor has signed CLA ready to pull Working to get PR submitted to internal repository, after which merging to Github happens.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants