[GitHub] [incubator-mxnet] arcadiaphy edited a comment on issue #15007: Add matrix determinant operator in linalg

[GitBox <gi...@apache.org>]

arcadiaphy edited a comment on issue #15007: Add matrix determinant operator in linalg
URL: https://github.com/apache/incubator-mxnet/pull/15007#issuecomment-495646192
 
 
   Updates:
   
   1. Only operators `det` and `slogdet` are kept in this PR.
   2. Zero gradient when det = 0 is intended behavior now and explained in operator docs.
   
   > There is no gradient backwarded when A is non-invertible (which is equivalent to det(A) = 0) because zero is rarely hit upon in float point computation and the Jacobi's formula on determinant gradient is not computationally efficient when A is non-invertible.
   

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