Skip to content

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

Open
glycerine wants to merge 5 commits intogorgonia:masterfrom
glycerine:auroc
Open

basic Mann-Whitney U statistic for area under ROC computation#400
glycerine wants to merge 5 commits intogorgonia:masterfrom
glycerine:auroc

Conversation

@glycerine
Copy link
Copy Markdown

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

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
Copy Markdown
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
Copy Markdown
Member

chewxy commented Apr 28, 2020

You beaut. :)

@owulveryck
Copy link
Copy Markdown
Member

Cool!
Thank you.

@glycerine
Copy link
Copy Markdown
Author

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

@chewxy
Copy link
Copy Markdown
Member

chewxy commented May 6, 2020

gonna start work on this today

@glycerine
Copy link
Copy Markdown
Author

gonna start work on this today

Sweet!

@glycerine
Copy link
Copy Markdown
Author

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

@chewxy
Copy link
Copy Markdown
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
Copy Markdown
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.

4 participants