-
Notifications
You must be signed in to change notification settings - Fork 16
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
add Lasso and MixedModels #25
Comments
See #27 for MixedModels. I'm more hesitant about Lasso as it's not very actively maintained. @AsafManela What do you think? |
I think including it is a pretty good way to incentivize more active maintenance :) |
IMHO, that's not how these things usually pan out. I often forget these bundle packages exist too :) |
Yes, don't count on it. :-) I think the question is rather whether the API is OK and well integrated with the current state of the ecosystem. (I haven't checked, but in particular there shouldn't be conflicting exports with other packages.) |
In that sense, I expect it to be fine. Lasso reuses a lot of GLM code and
mostly adheres to the StatsBase interface.
|
Have you considered adding Lasso and MixedModels to StatsKit?
The text was updated successfully, but these errors were encountered: