Unary ops logcdf_normal, derivlogcdf_normal [MXNET-39] #9982
Conversation
…ormal * New unary ops * Improved unit tests for basic unary, binary ops
Sorry, this is just #9961 reopened, I deleted the remote branch. |
Still not quite sure about the name... |
@piiswrong Are you worried about polluting the global namespace with specific mathematical functions? Any suggestions? Some thoughts:
These are really useful functions, MXNet should have them. Just let me know what to do. |
If things have to be moved to a namespace, I'd need help on how to do that: I don't know how this works. |
Is this exactly the same with scipy.stats.norm.logcdf? If so maybe we call it logcdf too? Does any other frameworks implement these functions? tf? |
Hi, the community has passed to vote about associating the code changes with JIRA (https://lists.apache.org/thread.html/ab22cf0e35f1bce2c3bf3bec2bc5b85a9583a3fe7fd56ba1bbade55f@%3Cdev.mxnet.apache.org%3E) We have updated the guidelines for contributors in https://cwiki.apache.org/confluence/display/MXNET/Development+Process, please ensure that you have created a JIRA at https://issues.apache.org/jira/projects/MXNET/issues/ to describe your work in this pull request and include the JIRA title in your PR as [MXNET-xxxx] your title where MXNET-xxxx is the JIRA id Thanks! |
Tensorflow has log_cdf in tf.distributions.Normal, see And yes: It is identical with scipy.stats.norm.logcdf. Your comments:
Note: These functions sound obscure, but they serve a large number of use cases in Bayesian ML. Some examples:
OK: How about we move this to contrib? And if nobody except us is using it much, it can stay there. Let me know. |
Hi Eric, Marco told me contrib is not a good place, and suggested to open a new namespace (say: math) for such more specific mathematical functions. They'd be out of the way of most MXNet users, and clunkier names should be more acceptable then. What do you think? I also wrote to dev@ |
Please make more efforts in filling out PR template. |
I think putting it in contrib is fine for now until we decide if we want a mx.distributions name space |
@mseeger ping |
I will close this request and reopen it later when I have more time to finish it properly |
New unary ops
Improved unit tests for basic unary, binary ops
Description
(Brief description on what this PR is about)
Checklist
Essentials
make lint
)Changes
Comments