-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Add L1 penalty and coordinate descent solver to TweedieRegression #16637
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
Comments
IIRC, one reason not to use the current |
@rth can I work on this? |
For users interested, in the meantime we have implement a Poisson datafit in the sklearn compatible Such datafit can be combined with a wealth of penalties (L1, but also WeightedL1, Elastic net - L1 + L2 -, and non convex penalties such as MCP, SCAD, Lq penalties with q < 1). We also have the Gamma datafit; feedback is welcome! |
The L1 penalty with coordinate descent solver for GLM was proposed in #9405 and was not included in the minimal implementation merged in #14300.
It would be be good to add it, however care must to taken to avoid redundancy with the existing coordinate descent solver in ElasticNet.
Preliminary benchmarks were done in #9405 (comment)
Also see: #16634 #16635
The text was updated successfully, but these errors were encountered: