You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/09/06 23:38:38 UTC

[GitHub] [incubator-mxnet] marcoabreu commented on a change in pull request #14535: [DOC] Updated install instructions for mac

marcoabreu commented on a change in pull request #14535: [DOC] Updated install instructions for mac
URL: https://github.com/apache/incubator-mxnet/pull/14535#discussion_r321941326
 
 

 ##########
 File path: docs/install/osx_setup.md
 ##########
 @@ -145,12 +155,16 @@ We have installed MXNet core library. Next, we will install MXNet interface pack
 To install the MXNet Python binding navigate to the root of the MXNet folder then run the following:
 
 ```bash
-$ cd python
-$ pip install -e .
+virtualenv -p`which python3` mxnet_py3
 
 Review comment:
   While I generally agree that virtualenv is a good practice, it adds another dependency.
   
   These guides are for the very basic user to get something running. I think this PR overcomplicates a lot of things. Adding the cmake compilation? Fine! Everything else? Please leave it as it is.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services