JulianSlzr commented on a change in pull request #9353: added mac gpu install; 
refactored windows install
URL: https://github.com/apache/incubator-mxnet/pull/9353#discussion_r162291214
 
 

 ##########
 File path: docs/install/index.md
 ##########
 @@ -709,28 +732,32 @@ Alternatively, you may follow the [CUDA installation 
instructions for Mac OS X](
 
 4. Install CUDA for MacOS X. Specific steps are provided in NVIDIA's [CUDA 
installation 
instructions](https://docs.nvidia.com/cuda/cuda-installation-guide-mac-os-x/index.html#installation).
 
-5. Run `git clone --recursive https://github.com/apache/incubator-mxnet.git 
mxnet` to get the latest version.
+5. 
[Download](http://docs.nvidia.com/deeplearning/sdk/cudnn-install/index.html#download-mac)
 and 
[install](http://docs.nvidia.com/deeplearning/sdk/cudnn-install/index.html#installmac)
 cuDNN for MacOS X. You will need to [create a free developer 
account](https://developer.nvidia.com/accelerated-computing-developer) with 
NVIDIA prior to getting the download link.
+
+**Step 4**  Build MXNet
+
+1. Run `git clone --recursive https://github.com/apache/incubator-mxnet.git 
mxnet` to get the latest version.
 
-6. Run `cd mxnet`.
+2. Run `cd mxnet`.
 
-7. Edit the `make/osx.mk` file to set the following parameters:
+3. Edit the `make/osx.mk` file to set the following parameters:
 
     ```
     USE_CUDA = 1
     USE_CUDA_PATH = /usr/local/cuda
     USE_CUDNN = 1
-    USE_OPENCV = 0
+    USE_OPENCV = 0   # set to 1 if you want to build with OpenCV
     ```
 
-8. Copy the `mxnet/make/osx.mk` to `mxnet/config.mk`
+4. Copy the `make/osx.mk` to `config.mk`
 
-9. Run `make`. If you previously attempted to compile you might want to do 
make clean_all first. You can also run `make -j` with the number of processors 
you have to compile with multithreading. There'll be plenty of warnings, but 
there should be no errors.
+5. Run `make`. If you previously attempted to compile you might want to do 
`make clean_all` first. You can also run `make -j` with the number of 
processors you have to compile with multithreading. There'll be plenty of 
warnings, but there should be no errors.
 
-10. Once finished, you should have a file called `libmxnet.so` in `mxnet/lib/`.
+6. Once finished, you should have a file called `libmxnet.so` in `lib/`.
 
-11. Do `cd python`.
+7. Do `cd python`.
 
-12. Run `sudo python setup.py install`.
+8. Run `sudo python setup.py install`.
 
 Review comment:
   Replace this with `sudo pip install -e .` (note that the `.` is part of the 
command). You could furthermore copy the blurb that says "Note that the -e flag 
is optional" etc. from the Linux instructions.

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


With regards,
Apache Git Services

Reply via email to