yngvem / group-lasso

Group Lasso implementation following the scikit-learn API
MIT License
105 stars 32 forks source link

Possible typo in error message #19

Closed SolidOrange closed 3 years ago

SolidOrange commented 4 years ago

Hello, I just got the following error message while trying to fit a LogisticGroupLasso:

RuntimeWarning: The FISTA iterations did not converge to a sufficient minimum.
You used subsampling then this is expected, otherwise,try to increase the number of iterations or decreasing the tolerance.

Shouldn't this recommend increasing the tolerance to reach convergence sooner?

yngvem commented 3 years ago

Thank you! This will be fixed in the next version