MRG Class weight refactor #1464
Conversation
|
|
sklearn/utils/__init__.py
Outdated
There was a problem hiding this comment.
This docstring deserves to better describe what's happening for the different values of the class_weight argument.
There was a problem hiding this comment.
Good point. Docstring coming tonight.
|
Renamed to MRG. Should be good now. |
|
More lines added then deleted :( Well, at least it is consistent with the other estimators now. that has been bothering me for a while. |
sklearn/svm/tests/test_svm.py
Outdated
|
I built this branch and the tests pass. +1 for merging once my last comment is addressed. |
sklearn/utils/__init__.py
Outdated
There was a problem hiding this comment.
This docstring is unclear. I think it should read
Array of class indices per sample; 0 <= y[i] < n_classes for i in range(n_samples).
|
@erg maybe you want to have a look as you complained so much about this ;) |
|
Rebased, changed the docstring. |
|
whoops, messed that one up... merged the wrong branch... hope no-one saw that ;) |
|
Ok so this is not merged but I can not reopen it. Great. |
Refactor
class_weightsfrom SGDClassifier and SVC, which enables the use ofclasses_in SVC.I put it in
utils.__init__, since I didn't know a where else to put it. Ideas welcome.Closes #745, #1037.