Line search - Githubissues

yngvem / group-lasso

Group Lasso implementation following the scikit-learn API

MIT License

105 stars 32 forks source link

Line search #21

Closed yngvem closed 3 years ago

yngvem commented 3 years ago

Added line search, preconditioning and fixed bugs

Features:

Backtracking line search for Lipschitz
Preconditioning via Centering

Bugfixes:

Chosen groups with same groups for each row
Intercept not used correctly for all logistic group lasso methods

pep8speaks commented 3 years ago

Hello @yngvem! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/group_lasso/_fista.py:

Line 23:80: E501 line too long (80 > 79 characters) Line 24:80: E501 line too long (80 > 79 characters) Line 25:80: E501 line too long (86 > 79 characters) Line 73:80: E501 line too long (87 > 79 characters) Line 77:80: E501 line too long (83 > 79 characters)

In the file src/group_lasso/_group_lasso.py:

Line 683:80: E501 line too long (83 > 79 characters) Line 684:80: E501 line too long (93 > 79 characters)

In the file test/test_group_lasso.py:

Line 410:80: E501 line too long (88 > 79 characters)

Comment last updated at 2020-07-18 11:48:23 UTC