mseeger commented on issue #9982: Unary ops logcdf_normal, derivlogcdf_normal 
for log CDF of standard n?
URL: https://github.com/apache/incubator-mxnet/pull/9982#issuecomment-370703827
 
 
   @piiswrong Are you worried about polluting the global namespace with 
specific mathematical functions?
   
   Any suggestions? Some thoughts:
   - Could move them to random namespace, as this is about functions related to 
random variates.
      Drawback is that random so far has only sampling ops
   - A new namespace, say stats? Drawback: Not much in there
   - An operator called "normal_funcs" with a string arg, which I can set to 
"logcdf", "deriv_logcdf".
      This is just a bit harder to implement, cannot use the default macros for 
unary ops
   - Move to contrib namespace. I'd be OK with this, but note this is just as 
solid as gamma or
      gammaln, there is nothing experimental about this code. I have the 
feeling that stuff in
      contrib is not generally trusted
   
   These are really useful functions, MXNet should have them. Just let me know 
what to do.
   

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