-
Notifications
You must be signed in to change notification settings - Fork 33
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
[WIP] sklearn-like LogisticRegression #99
Conversation
@agramfort I don't support multinomial Logreg and |
I just looked at estimator tags we have now in sklearn and you cannot limit
to binary classification.
one option is to wrap the estimator using OneVsRestClassifier metaestimator
… |
That did the trick @agramfort: tests are convinving. only remains to write some doc + example. |
Codecov Report
@@ Coverage Diff @@
## master #99 +/- ##
==========================================
+ Coverage 68.95% 71.21% +2.26%
==========================================
Files 10 10
Lines 612 667 +55
Branches 91 93 +2
==========================================
+ Hits 422 475 +53
Misses 166 166
- Partials 24 26 +2
Continue to review full report at Codecov.
|
great !
… |
Codecov Report
@@ Coverage Diff @@
## master #99 +/- ##
==========================================
+ Coverage 68.95% 71.21% +2.26%
==========================================
Files 10 10
Lines 612 667 +55
Branches 91 93 +2
==========================================
+ Hits 422 475 +53
Misses 166 166
- Partials 24 26 +2
Continue to review full report at Codecov.
|
#85