[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372565487 > Not sure. At this point, I really don't know what's going on with the mkl build, what works with what, does mkldnn use mklml or mkl? can mkldnn operate on its own, what combinations work, etc. @cjolivier01 MKL-DNN can work standalone (you can build it without MKL/MKLML and it will perform great). MKL-DNN uses MKL/MKLML only to accelerate inner product via gemm from MKL/MKLML. MKLML is a small library, which contains part of MKL functionality (ML related). **update:** information from MKL-DNN team for MKLML. > MKLML includes all single precision BLAS functions, a subset of vector math functions, and a subset of LAPACK. The library is created to support self-contained distribution of open source deep learning applications and for this scenario can be used instead of full MKL. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372943647 @KellenSunderland Agree, this change is good. We can merge it. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372943647 @KellenSunderland Agree, this change is good. We can merge it. - when full MKL is not installed, MKLML can provide the basic BLAS for MKLDNN. And openBLAS (other BLAS) can be used for other basic GEMM operations. - when full MKL is installed, the MKLML is not needed because it's only a part of full MKL. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372868894 @anirudh2290 it's in the MKL-DNN readme by "Intel MKL small libraries" and I am working on improving the explanations. I will update these descriptions into MXNET doc later. https://github.com/intel/mkl-dnn/blob/master/README.md#installation @KellenSunderland I am looking the whole logic. And as @cjolivier01 mentioned, the current building logic is not very clear. The CMake and Makefile flow is not consistency. So, I am asking @jinhuang415 to write up a proposal to make a clear building approach of MKL/MKL-DNN/MKLML. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372868894 @anirudh2290 it's in the MKL-DNN readme by "Intel MKL small libraries" and I am working on improving the explanations. I will update these descriptions into MXNET doc later. https://github.com/intel/mkl-dnn/blob/master/README.md#installation @KellenSunderland I am looking the whole logic. And as @cjolivier01 mentioned, the current building logic is not very clear. The CMake and Makefile flow is not consistency. So, I am asking @jinhuang415 to write up a proposal to make a clear building approach of MKL/MKL-DNN/MKLML. 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: us...@infra.apache.org With regards, Apache Git Services
[GitHub] pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL.
pengzhao-intel commented on issue #10075: [MXNET-83] Fix CMake build issue with MKL. URL: https://github.com/apache/incubator-mxnet/pull/10075#issuecomment-372868894 @anirudh2290 it's in the MKL-DNN readme by "Intel MKL small libraries" and I am working on improving the explanations. I will update these descriptions into MXNET doc later. https://github.com/intel/mkl-dnn/blob/master/README.md#installation @KellenSunderland I am looking the changes and try to understand the logic. Will back to you soon. 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: us...@infra.apache.org With regards, Apache Git Services