marcoabreu commented on issue #10921: Test cases improvement for MKLDNN on Gluon
URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-392412662
 
 
   I think they were originally introduced to test int32 overflow, but I
   totally agree that these are suboptimal and should be rather be mocked in
   the C++ tests opposed to actually allocating so much memory.
   
   Da Zheng <notificati...@github.com> schrieb am Mo., 28. Mai 2018, 04:35:
   
   > @marcoabreu <https://github.com/marcoabreu> I think we should remove the
   > tests for generating huge weight matrices, as @juliusshufan
   > <https://github.com/juliusshufan> is doing right now for Dense.
   > It's clear that the code will run into out-of-memory errors, regardless of
   > the backend (native MXNet, MKLDNN or CuDNN).
   >
   > —
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > 
<https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-392403691>,
   > or mute the thread
   > 
<https://github.com/notifications/unsubscribe-auth/ARxB65K-DpBIA66UA8wL2SxlG9xIEjWWks5t22JzgaJpZM4T8smV>
   > .
   >
   

----------------------------------------------------------------
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

Reply via email to