-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] add balanced_accuracy_score metric #3506 #3929
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
Oh, so should I close mine PR? There are still some TODOs in my PR that I haven't finished though. |
y_type, y_true, y_pred = _check_targets(y_true, y_pred) | ||
|
||
if y_type != "binary": | ||
raise ValueError("%s is not supported" % y_type) |
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.
Should multilabel be supported?
Would be nice to add an example illustrating possible use-cases of balanced accuracy. For instance a binary classification task with highly imbalanced classes, where a classifier which assigns everything to the majority class obtains high accuracy but low balanced-accuracy. |
I'm not sure if the documentation has a similar example for P/R/F, but an On 14 December 2014 at 23:45, Jan Hendrik Metzen notifications@github.com
|
I think adding an example would be a good idea. I will work on the comment by @jnothman and then on an example in a couple of weeks during the christmas break! |
or label indicator array / sparse matrix Predicted labels, as returned | ||
by a classifier. | ||
|
||
c : is the cost associated with the misclassification of a positive example. |
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.
no one-letter parameters ;)
The other PR says |
@amueller No. I don't think so. For example,
The wikipedia definition of jaccard similarity http://en.wikipedia.org/wiki/Jaccard_index#Similarity_of_asymmetric_binary_attributes |
closed in favor of #5588 |
It refers to the issue #3506