-
Notifications
You must be signed in to change notification settings - Fork 2
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
Relax primal-dual system upon linesearch failure #8
Conversation
I thought I had tested something like this and concluded that it wasn't enough because it cycles. Maybe try my MVP from #7 as a test? |
I think the |
M is different: |
Right... helps to read ;) So that |
Ah I see - basically the quadratic term will evaluate to zero for all x. Then the well posedness entirely depends on the sign of theta. |
test/runtests.jl
Outdated
@@ -14,9 +14,10 @@ using FiniteDiff: FiniteDiff | |||
0 ≤ y ⟂ H(x, y) = Ax - b ≥ 0. | |||
""" | |||
M = [2 1; 1 2] | |||
# M = [0 1; -1 0] |
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.
This is never used. Intended?
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.
Good catch
Adjusts
\epsilon
as follows:\epsilon *= 1 - exp(-iters)
\epsilon *= 1 + exp(-iters)
This fixes #7.