Skip to content

Conversation

@yifeif
Copy link
Contributor

@yifeif yifeif commented Dec 10, 2016

No description provided.


If the above commands do not work on your system or you want to install the GPU version
of the binary on Linux or Mac, you can follow these instructions:
Note that you will need pip version 8.1 or later for the above commands to work on Linux.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's have this at the top, as one of the requirements.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Decided to not add it as a requirement as this is only needed for 'pip install tensorflow' to work. If for some reason users cannot upgrade pip, I dont want them to think they cannot install tensorflow. What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

It should still appear at the top of this section. IMO people read until they have the minimum information they need, and stop there. They wont read it where it is now at the bottom.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what about now?

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good, thanks!

@andrewharp andrewharp merged commit 68322a6 into tensorflow:r0.12 Dec 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants