Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

basic Mann-Whitney U statistic for area under ROC computation #400

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

glycerine
Copy link

Hi @chewxy, here's the fundamental area under the ROC curve computation code -- as mentioned in our twitter discussion. The computation is standalone correct but I didn't attempt the integration with the existing library. I place it into the public domain so it can be re-used in any context. - Jason

@coveralls
Copy link

coveralls commented Apr 27, 2020

Coverage Status

Coverage decreased (-0.07%) to 64.085% when pulling f2f272c on glycerine:auroc into 5fb5944 on gorgonia:master.

@glycerine
Copy link
Author

here's one approach to making it differentiable for optimization

http://citeseerx.ist.psu.edu/viewdoc/summary?doi=10.1.1.18.8531

@chewxy
Copy link
Member

chewxy commented Apr 28, 2020

You beaut. :)

@owulveryck
Copy link
Member

Cool!
Thank you.

@glycerine
Copy link
Author

I added an option to compute the approximation function for comparison.

@chewxy
Copy link
Member

chewxy commented May 6, 2020

gonna start work on this today

@glycerine
Copy link
Author

gonna start work on this today

Sweet!

@glycerine
Copy link
Author

Let me know if there's testing or anything I can help with.

@chewxy
Copy link
Member

chewxy commented May 9, 2020

Sorry, I was working on this and realized it's quite good - and got side tracked. The code is now placed into https://github.com/gorgonia/qol/blob/master/metrics/auc.go...

The differentiable Op version for this is coming...

@chewxy
Copy link
Member

chewxy commented May 9, 2020

Moving to #401 for the Op/differentiable version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants