litaotju edited a comment on issue #17373: Fix mshadow broken basic_stream 
example
URL: https://github.com/apache/incubator-mxnet/pull/17373#issuecomment-575969191
 
 
   > Any tests to confirm the change?
   
   @ChaiBapchya 
   Tested with the following edited config.mk. The sample only compiled when 
`USE_CUDA=1`.
   Since not all user use CUDA, I didn't make it on-by-default.
   ```
   diff --git a/3rdparty/mshadow/guide/config.mk 
b/3rdparty/mshadow/guide/config.mk
   index 26431bb06..9ebc4f685 100644
   --- a/3rdparty/mshadow/guide/config.mk
   +++ b/3rdparty/mshadow/guide/config.mk
   @@ -13,21 +13,22 @@
    
#----------------------------------------------------------------------------------------
   
    # whether use CUDA during compile
   -USE_CUDA = 0
   +USE_CUDA = 1
   
    # add the path to CUDA libary to link and compile flag
    # if you have already add them to enviroment variable, leave it as NONE
   -USE_CUDA_PATH = NONE
   +USE_CUDA_PATH = /usr/local/cuda
   
    #
    # choose the version of blas you want to use
    # can be: mkl, blas, atlas, openblas, apple
   -USE_BLAS = blas
   +USE_BLAS = mkl
    #
    # add path to intel library, you may need it
    # for MKL, if you did not add the path to enviroment variable
    #
   -USE_INTEL_PATH = NONE
   +USE_INTEL_PATH = /opt/intel/
   +USE_STATIC_MKL = NONE
   
    # whether compile with parameter server
    USE_DIST_PS = 0
   (END)
   ```
   ---
   Complete config.mk is here.
   https://gist.github.com/litaotju/5954a59cf9d4f4b0f3daf22a58794426

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to