Skip to content
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

Documentation on convex solvers confusing #266

Open
rileyjmurray opened this issue Aug 24, 2018 · 4 comments
Open

Documentation on convex solvers confusing #266

rileyjmurray opened this issue Aug 24, 2018 · 4 comments

Comments

@rileyjmurray
Copy link

One page says that Elemental's convex solvers are based on the Mehrotra predictor–corrector method.

But another page says they are based on ADMM.

ADMM and Mehrotra's predictor-corrector method are very different. Indeed, both are primal-dual solvers, but ADMM is considered "first order" while the predictor-corrector method is "second order". It seems to me that the sentence

Elemental currently supports distributed dense and sparse Linear, Quadratic, and Second-Order Cone Programs via Mehrotra Predictor-Corrector primal-dual Interior Point Methods

from the first link should simply be updated to

Elemental currently supports distributed dense and sparse Linear, Quadratic, and Second-Order Cone Programs via ADMM.

@poulson
Copy link
Member

poulson commented Aug 24, 2018

Hi Riley,

There is indeed code for primal dual IPMs, though the stability of the numerics across the range of precisions is something I stopped working on.

@poulson
Copy link
Member

poulson commented Aug 24, 2018

@poulson
Copy link
Member

poulson commented Aug 24, 2018

Again, the numerics is in flux (nearly resolved even for infinite precision in a branch), but I haven't worked on this project in nearly two years.

@rileyjmurray
Copy link
Author

I see. Thank you for the explanation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants