You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/07/19 18:55:14 UTC

[GitHub] [incubator-mxnet] kshitij12345 edited a comment on issue #15611: [MXNET-978] n-th order gradient test support

kshitij12345 edited a comment on issue #15611: [MXNET-978] n-th order gradient test support
URL: https://github.com/apache/incubator-mxnet/pull/15611#issuecomment-513333644
 
 
   @apeforest @larroy 
   
   While working on a PR (I don't exactly remember which one), test for incorrect implementation of second order gradient was passing due to the very small values. However as sanity check, when I checked for its third order gradient, the assertion failed allowing to catch the issue. 
   
   So I think it would be better to check for the order being implemented and one more (computed by autograd) as a sanity check. 
   
   Would like to know your thoughts about the same. 
   
   Thank You.
   
   

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


With regards,
Apache Git Services