[Bug] ADMM is never converged

[JakobAsslaender]

@nHackel:

I am not sure if I am overlooking something, but you changed the output of the ADMM converged function to always return false:

RegularizedLeastSquares.jl/src/ADMM.jl

Line 284 in 06c8405

return false

I presume this is a bug and should be reverted to return true after the for loop, if it gets there, right?

[nHackel]

Oh yes, that is definitely not intentional 🙈

[nHackel]

I'll fix that on the master real quick

[nHackel]

Maybe we should add convergence tests. Just a small problem with an "excessive" number of iterations and then we check afterwards if it converged