Fix bug in admm.LinearSubproblemSolver

[bwohlberg]

Fix bug in admm.LinearSubproblemSolver, resolving #90.

[codecov]

Codecov Report

Merging #91 (70a041c) into main (4684909) will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
+ Coverage   90.17%   90.37%   +0.20%     
==========================================
  Files          42       42              
  Lines        2981     2981              
==========================================
+ Hits         2688     2694       +6     
+ Misses        293      287       -6     

Flag	Coverage Δ
unittests	90.37% <100.00%> (+0.20%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
scico/admm.py	89.79% <100.00%> (+1.02%)	⬆️
scico/loss.py	87.93% <0.00%> (+3.44%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4684909...70a041c. Read the comment docs.

[smajee]

This fixes the error, but when I run it, the solution looks weird to me.

[bwohlberg]

This fixes the error, but when I run it, the solution looks weird to me.

Resolved after fixing the error pointed out by @tbalke.

[bwohlberg]

Converted to draft to avoid accidental merge: this PR likely to be discarded since its branch has been merged into #88.

[bwohlberg]

Closing since all changes merged into mike/approx_prox, the branch for #88.