-
Notifications
You must be signed in to change notification settings - Fork 34
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
ENH - Add modular Group Prox Newton solver #103
ENH - Add modular Group Prox Newton solver #103
Conversation
…into group-pn-solver
For the benchmarks, |
@Badr-MOUFAD the cross entropy for 2 classes should be the logistic loss |
Pls have a look at this potion the code. Also in the benchmark, we don't converge to the same objective. |
This is what one would expect. Also, after checking, yngvem I raised an issue yngvem/group-lasso#36. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First round of review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for the outstanding work @Badr-MOUFAD !
One last question, GroupProxNewton
has not been added to all the tests in the test_group.py
file, is there a specific reason?
It's mainly to avoid redundancy and complications of unittest:
|
This implements a group proximal Newton solver to extend
skglm
to handle problems with non-quadratic datafit and group separable penalties.It proceeds as follows:
GroupProxNewton
link to benchmarks results
link to benchmark repo