matteosal edited a comment on issue #16555: Upgrade MKL-DNN dependency to v1.0 URL: https://github.com/apache/incubator-mxnet/pull/16555#issuecomment-546457096 @pengzhao-intel > @matteosal this is really helpful for us :) > How many issues do you see until now? > How about the performance on your side with the new MKL-DNN? I see several test failures, but I expect many of them are coming fom the same underlying issue and others are due to numerical checks failing due to the change of backend. Unfortunately our wrappers are quite complicated, so it takes a non trivial amount of time for me to isolate the sequence of mxnet calls triggering the error and building an equivalent python example. In addition, for the next two weeks I'll be quite busy wrapping up new features for the new Wolfram Language release, so in the next days I will isolate and report one issue at a time. @TaoLv > @matteosal Thank you for reporting. Have you ever tried this case with the latest master branch? I've just tried now, and it also fails on master indeed. But it still comes from MKL-DNN, because it goes away if I disable the backend via env var. Should I report these kind of failures here? > To help us to isolate the issue, could you please comment out below lines from src/operator/tensor/matrix_op.cc and then build mxnet and test your case again? It doesn't crash with these changes
---------------------------------------------------------------- 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
