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

update status_mapt to latest constants #66

Closed
wants to merge 1 commit into from
Closed

update status_mapt to latest constants #66

wants to merge 1 commit into from

Conversation

kalmarek
Copy link
Collaborator

Does SCS_SOLVED_INACCURATE qualify for :UserLimit?

Does `SCS_SOLVED_INACCURATE` qualify for `:UserLimit`?
@mlubin
Copy link
Member

mlubin commented Feb 14, 2017

What are the conditions that could trigger that status? If it's an iteration limit, then yes, I'd say UserLimit.

By the way, we're thinking about reworking the MathProgBase status system because it's pretty broken at the moment, see the discussion at JuliaOpt/MathProgBase.jl#156.

@kalmarek
Copy link
Collaborator Author

Yest this can be an iteration limit

@mlubin
Copy link
Member

mlubin commented Feb 14, 2017

UserLimit isn't appropriate if the solver stops because of an iteration limit which the user didn't set.

@kalmarek
Copy link
Collaborator Author

kalmarek commented Apr 1, 2017

I'm pretty sure this is me being dumb, but I have no idea how to distinguish SCSSolver() and SCSSolver(iterations=20000) in code; for the first hitting its default 20000 iterations shouldn't return :UserLimit, while the second should (then what should the first one return? :Inaccurate?)

How is discussion on extending the number of solve states progressing?

@mlubin
Copy link
Member

mlubin commented Apr 1, 2017

User-provided options are stored in SCSMathProgModel.options, so you can check if max_iters was set. If SCS stops because it hit the user-provided iteration limit, it should return :UserLimit. If it stops because it hits its internal limit without solving the problem, it should return something like :Error.

I wouldn't expect a revision of the MPB solver statuses until June.

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

Successfully merging this pull request may close these issues.

2 participants