-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] implement balanced_accuracy_score #4300
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
Conversation
any comment @arjoly @adam-m-mcelhinney @jnothman @larsmans @ogrisel ? |
8b579ca
to
fdda785
Compare
Returns | ||
------- | ||
C : array, shape = [n_classes, n_classes] | ||
Confusion matrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not what is returned by this function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is my very first PR. I will rethink the code. Thanks! @ogrisel
This function needs to be referenced in the documentation where appropriate. |
fdda785
to
0cb25f8
Compare
Thanks @ogrisel! Fixed the problems you commented, but I don't it is necessary to change |
0cb25f8
to
de84a06
Compare
closed in favor of #5588 |
Added balanced_accuracy_score. #3506
My implementation is very simple. The definition of balanced accuracy score is
0.5 * true positives / (true positives + false negatives) + 0.5 * true negatives / (true negatives + false positives)
It is actually the average of positive label recall and the negative label recall. It supports only binary label situation now.
There are other two relevant PRs #3929 #3511.