8000 Update LogisticRegression table summary to mention l-bfgs default · Issue #12617 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

Update LogisticRegression table summary to mention l-bfgs default #12617

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.

Sign up for GitHub

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

Jump to bottom
Closed
amueller opened this issue Nov 19, 2018 · 2 comments · Fixed by #12735
Closed

Update LogisticRegression table summary to mention l-bfgs default #12617

amueller opened this issue Nov 19, 2018 · 2 comments · Fixed by #12735
Labels
Documentation Easy Well-defined and straightforward way to resolve

Comments

@amueller
Copy link
Member

We should update the summary of the table here to mention l-bfgs and it's benefits/downsides as this is our future default.
Probably after we merged #11646.
@amueller amueller added Easy Well-defined and straightforward way to resolve Documentation help wanted labels Nov 19, 2018
@hassaanseeker
Copy link
Contributor

Hello amueller i would like to work on this. And update the documentation.

We should update the summary of the table here to mention l-bfgs and it's benefits/downsides as this is our future default.
Probably after we merged #11646.

@TomDLT
Copy link
Member
TomDLT commented Nov 22, 2018

Cool, go ahead !

@hassaanseeker hassaanseeker mentioned this issue Nov 23, 2018
Closed
@hassaanseeker hassaanseeker mentioned this issue Dec 9, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation Easy Well-defined and straightforward way to resolve
Projects
None yet
Milestone
No milestone
4 participants
@amueller @adrinjalali @hassaanseeker @TomDLT
0