[GitHub] [incubator-mxnet] sxjscience commented on issue #17628: Missing dnnl_config.h when installing horovod

2020-02-19 Thread GitBox
sxjscience commented on issue #17628: Missing dnnl_config.h when installing 
horovod
URL: 
https://github.com/apache/incubator-mxnet/issues/17628#issuecomment-588600364
 
 
   That’s currently the only one I found for linking to horovod. However I just 
ran setup.py in horovod and hasn’t run distributed training yet. I’m having 
some other issues in the cmake-built MXNet which I’m still investigating.
   
   Get Outlook for iOS
   
   From: Tao Lv 
   Sent: Wednesday, February 19, 2020 5:15:05 PM
   To: apache/incubator-mxnet 
   Cc: Xingjian SHI ; Mention 

   Subject: Re: [apache/incubator-mxnet] Missing dnnl_config.h when installing 
horovod (#17628)
   
   
   I will take a look at this. I remember previously the recommended build 
system of mxnet was makefile when cooperating with horovod.
   @sxjscience is this the only issue when using 
cmake to build mxnet and linking it to horovod?
   
   —
   You are receiving this because you were mentioned.
   Reply to this email directly, view it on 
GitHub,
 or 
unsubscribe.
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-mxnet] sxjscience commented on issue #17628: Missing dnnl_config.h when installing horovod

2020-02-19 Thread GitBox
sxjscience commented on issue #17628: Missing dnnl_config.h when installing 
horovod
URL: 
https://github.com/apache/incubator-mxnet/issues/17628#issuecomment-588406327
 
 
   A temporary fix is to manually copy files inside the `build` directory to 
`include/mkldnn`. I've confirmed that this solves the problem in horovod 
installation.
   ```
   cp build/3rdparty/mkldnn/include/dnnl_config.h include/mkldnn
   cp build/3rdparty/mkldnn/include/dnnl_version.h include/mkldnn
   ```
   


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:
us...@infra.apache.org


With regards,
Apache Git Services