10000 [MRG+1] Removed wrong warning message about alpha in ElasticNet by Sentient07 · Pull Request #8565 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

[MRG+1] Removed wrong warning message about alpha in ElasticNet #8565

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
Merged
merged 1 commit into from
Mar 17, 2017
Merged

[MRG+1] Removed wrong warning message about alpha in ElasticNet #8565

merged 1 commit into from
Mar 17, 2017

Conversation

Sentient07
Copy link
Contributor

Fixes #8233

@NelleV
Copy link
Member
8000 NelleV commented Mar 17, 2017

LGTM 👍
Thanks for the patch!

@NelleV NelleV changed the title Removed wrong warning message about alpha in ElasticNet [MRG+1] Removed wrong warning message about alpha in ElasticNet Mar 17, 2017
@lesteve
Copy link
Member
lesteve commented Mar 17, 2017

LGTM, merging, thanks a lot!

@lesteve lesteve merged commit eba8a6a into scikit-learn:master Mar 17, 2017
@Przemo10 Przemo10 mentioned this pull request Mar 17, 2017
Closed
herilalaina pushed a commit to herilalaina/scikit-learn that referenced this pull request Mar 26, 2017
@Sentient07 @herilalaina
Improve warning message in ElasticNet (scikit-learn#8565)
df277eb
massich pushed a commit to massich/scikit-learn that referenced this pull request Apr 26, 2017
@Sentient07
Improve warning message in ElasticNet (scikit-learn#8565)
b90aecb
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
@Sentient07 @Sundrique
Improve warning message in ElasticNet (scikit-learn#8565)
80ec47b
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
@Sentient07 @NelleV
Improve warning message in ElasticNet (scikit-learn#8565)
671963b
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
@Sentient07 @paulha
Improve warning message in ElasticNet (scikit-learn#8565)
34349c2
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
@Sentient07 @maskani-moh
Improve warning message in ElasticNet (scikit-learn#8565)
79a7f26
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
@Sentient07
Improve warning message in ElasticNet (scikit-learn#8565)
58dc3e6
@jonathan-taylor
Copy link

A bit perplexed by this message. With l1_ratio=0 and alpha>0 the problem is strongly convex and separable. Coordinate descent will converge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Sentient07 @NelleV @lesteve @jonathan-taylor
0