Hi Pedro, Thanks for the good suggestions. I plan to update the install page but a slight delay by other things.
Actually, I didn't test the performance on OSX since we're lack of Mac machine. I will try if I can get a Mac in the office. Thanks, --Patric > -----Original Message----- > From: Pedro Larroy [mailto:pedro.larroy.li...@gmail.com] > Sent: Wednesday, March 14, 2018 9:09 PM > To: dev@mxnet.incubator.apache.org > Subject: Re: call for contributions to next MXNet release > > Hi Patric > > Does it make sense to add instructions to link against mkl in the install > docs? > > For example in OSX you can use MKL by downloading > https://github.com/intel/mkl-dnn/releases and adding MKL_RT_LIBRARY and > MKL_INCLUDE_DIR plus -DUSE_MKL_IF_AVAILABLE=ON - > DUSE_MKLML_MKL=ON -DUSE_MKLDNN=ON > > This is not documented in the website. > > What's the performance increase of using MKL in osx when running in CPU > mode? > > Pedro > > On Wed, Mar 14, 2018 at 1:54 PM, Zhao, Patric <patric.z...@intel.com> > wrote: > > > My fault, typo for your name, Larroy :( > > > > > -----Original Message----- > > > From: Zhao, Patric [mailto:patric.z...@intel.com] > > > Sent: Wednesday, March 14, 2018 8:40 PM > > > To: dev@mxnet.incubator.apache.org; Huang, Jin1 > > > <jin1.hu...@intel.com> > > > Subject: RE: call for contributions to next MXNet release > > > > > > Thanks, Larryo. > > > > > > This is a PR to fix this issue, https://github.com/apache/incubator- > > > mxnet/pull/10075 > > > The issue will be gone after this PR is merged. > > > > > > For the whole logic, we are investigating now and maybe propose a > > > new approach for Makefile and CMake. > > > > > > > -----Original Message----- > > > > From: Pedro Larroy [mailto:pedro.larroy.li...@gmail.com] > > > > Sent: Wednesday, March 14, 2018 8:31 PM > > > > To: dev@mxnet.incubator.apache.org > > > > Subject: Re: call for contributions to next MXNet release > > > > > > > > Hi > > > > > > > > Can you please fix the build logic? > > > > https://github.com/apache/incubator- > > > > mxnet/blob/master/CMakeLists.txt#L158 > > > > > > > > https://github.com/apache/incubator-mxnet/issues/10072 > > > > > > > > It wrongly assumes that you have MKL installed, MKLDNN needs to > > > > check if MKL is available before, or be disabled by default, also > > > > in non > > intel > > > platforms. > > > > > > > > Pedro > > > > > > > > On Sun, Mar 11, 2018 at 2:43 PM, Zhao, Patric > > > > <patric.z...@intel.com> > > > > wrote: > > > > > > > > > Steffen, really thanks for the helps and I totally agree with > > > > > you and Marco's suggestions. > > > > > I will summarize the open issues and status for the review soon. > > > > > > > > > > > What is your github handle? > > > > > Github ID: Patric Zhao, https://github.com/pengzhao-intel/ Da > > Zheng, > > > > > https://github.com/zheng-da/ > > > > > > > > > > > - do you have design docs we can link to? Does the doc cover > > > > > > packaging, > > > > > use > > > > > > of MXNet with full MKL (I understand there are license issues, > > > > > > but we do > > > > > have > > > > > > users who are or plan to use MXNet with the complete MKL > > > > > > package, not > > > > > just > > > > > > DNN subset)? > > > > > Design doc: > > > > > https://cwiki.apache.org/confluence/display/MXNET/The+ > > > > > design+of+MKLDNN+integration > > > > > It's a good suggestion for the full MKL package and we will add > > > > > the related part in the doc. > > > > > > > > > > > - do you have performance measurements (or plan to measure) to > > > > > > include in release notes? > > > > > Yes, we have tested the performance on all C4/C5 instances > > > > > (8x,4x,2x,xlarge,large). And the accuracy are verified as well. > > > > > We plan to publish these data both in the release note and mxnet > > > > > website ( https://mxnet.incubator.apache.org/faq/perf.html). > > > > > > > > > > > - should we talk about the package at the Apr 24th meetup in > > Seattle? > > > > > We're based in Shanghai, China; so maybe @Da Zheng? > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: Steffen Rochel [mailto:steffenroc...@gmail.com] > > > > > > Sent: Sunday, March 11, 2018 6:11 PM > > > > > > To: dev@mxnet.incubator.apache.org > > > > > > Cc: marco.g.ab...@googlemail.com > > > > > > Subject: Re: call for contributions to next MXNet release > > > > > > > > > > > > Patric - added MKL-DNN to the project list. What is your > > > > > > github > > handle? > > > > > > I do agree with Marco that we need to resolve disable tests, > > > > > > broken > > > > > features > > > > > > etc. Jira looks like the right answer to create a list of > > > > > > known > > issues. > > > > > > Questions I do have: > > > > > > - do you have design docs we can link to? Does the doc cover > > > > > > packaging, > > > > > use > > > > > > of MXNet with full MKL (I understand there are license issues, > > > > > > but we do > > > > > have > > > > > > users who are or plan to use MXNet with the complete MKL > > > > > > package, not > > > > > just > > > > > > DNN subset)? > > > > > > - do you have performance measurements (or plan to measure) to > > > > > > include in release notes? > > > > > > - should we talk about the package at the Apr 24th meetup in > > Seattle? > > > > > > > > > > > > Steffen > > > > > > > > > > > > On Sat, Mar 10, 2018 at 4:40 AM Zhao, Patric > > > > > > <patric.z...@intel.com> > > > > > > wrote: > > > > > > > > > > > > > Hi Marco, > > > > > > > > > > > > > > Thanks for the inputs. > > > > > > > > > > > > > > MKL-DNN is just merged to the master branch in a month. I > > > > > > > agree that it's not very mature. > > > > > > > But, in my mind, there're NO major issues in the current > > > > > implementation. > > > > > > > > > > > > > > The previous data race (flaky test) issue is fixed by Zheng Da. > > > > > > > We have submitted the PRs to fix the building issues and > > > > > > > setup Clang CI environment, such as in OSX and Cmake. > > > > > > > Meanwhile, we are actively working on the MXNET for the > > > > > > > performance, functionality and usability improvements. > > > > > > > > > > > > > > More positively, I think we should summarize the open > > > > > > > issues, and we can focus on these issues you've mentioned. > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > --Patric > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > From: Marco de Abreu [mailto:marco.g.ab...@googlemail.com] > > > > > > > > Sent: Saturday, March 10, 2018 9:36 AM > > > > > > > > To: dev@mxnet.incubator.apache.org > > > > > > > > Subject: Re: call for contributions to next MXNet release > > > > > > > > > > > > > > > > Hello Patric, > > > > > > > > > > > > > > > > please be aware of the fact that there are still a lot of > > > > > > > > disabled > > > > > > > tests, open > > > > > > > > MKLDNN issues, broken features and flaky tests that have > > > > > > > > not been addressed yet. We agreed on merging MKLDNN for > > > > > > > > the time being to allow broad testing and that we will > > > > > > > > revisit the state a bit before the next > > > > > > > release. > > > > > > > > At the moment, I'd not be in favour of having MKLDNN being > > > > > > > > part of > > > > > it. > > > > > > > > > > > > > > > > Best regards, > > > > > > > > Marco > > > > > > > > > > > > > > > > Zhao, Patric <patric.z...@intel.com> schrieb am Sa., 10. > > > > > > > > März 2018, > > > > > > > 02:30: > > > > > > > > > > > > > > > > > Hi Steffen, > > > > > > > > > > > > > > > > > > We'd like the MKL-DNN backend can be included in the > > > > > > > > > next > > > release. > > > > > > > > > > > > > > > > > > We (Intel engineers) and Zheng-Da (AWS engineer) can > > > > > > > > > work > > on it. > > > > > > > > > > > > > > > > > > Could you help add the item in the table? > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > > > > --Patric > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > From: Steffen Rochel [mailto:steffenroc...@gmail.com] > > > > > > > > > > Sent: Friday, March 9, 2018 9:29 PM > > > > > > > > > > To: dev@mxnet.incubator.apache.org > > > > > > > > > > Subject: call for contributions to next MXNet release > > > > > > > > > > > > > > > > > > > > Hi - I would like to propose the next MXNet release. > > > > > > > > > > Initial draft content is listed at MXNet wiki > > > > > > > > > > <https://cwiki.apache.org/confluence/display/MXNET/Pro > > > > > > > > > > ject > > > > > > > > > > +P > > > > > > > > > > ropo > > > > > > > > > > sals > > > > > > > > > > +fo > > > > > > > > > > r+next+MXNet+Release>. > > > > > > > > > > I would like to call to all contributors to add > > > > > > > > > > features you are working > > > > > > > > > on and > > > > > > > > > > would like to see included in the release. Suggested > > > > > > > > > > code freeze is March 30th with target release by mid April. > > > > > > > > > > > > > > > > > > > > Anirudh Subramanian and Chris Olivier have volunteered > > > > > > > > > > to co-manage the release. > > > > > > > > > > > > > > > > > > > > Mark your date: we are planing public meetup on Apr > > > > > > > > > > 24th in > > > > > Seattle. > > > > > > > > > > Details to follow. > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > Steffen > > > > > > > > > > > > > > > > > > > > > > >
