Skip to content

Conversation

odow
Copy link
Member

@odow odow commented May 27, 2021

Closes #1372

@odow odow added the Type: Documentation This issue requires changes to the documentation label May 27, 2021
@odow odow force-pushed the od/order_of_solutions branch from ad58ee4 to fbd3903 Compare May 27, 2021 21:27
@blegat
Copy link
Member

blegat commented May 28, 2021

If multiple results are present, the first result (i.e., result_index=1) must be an optimal solution.

This disallows having multiple results in case the termination status is not OPTIMAL.
I would rather say something like

If the termination status is OPTIMAL, and multiple results are present, the first result (i.e., result_index=1) must be an optimal solution.
If the termination status is INFEASIBLE and one of the results is an infeasibility certificate, it should be first result.
If the termination status is DUAL_INFEASIBLE and one of the results is a dual infeasibility certificate, it should be the first result.
If the termination status is LOCALLY_SOLVED and one of the results is a stationary point or local optimal solution, it should be the first result.

At least that's what we assume in the tests so it makes sense to make it a requirements.

@odow odow force-pushed the od/order_of_solutions branch from fbd3903 to 64c4d9a Compare May 28, 2021 23:20
@odow
Copy link
Member Author

odow commented May 29, 2021

How's this now.

@odow odow merged commit 65924e3 into master May 31, 2021
@odow odow deleted the od/order_of_solutions branch May 31, 2021 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation This issue requires changes to the documentation
Development

Successfully merging this pull request may close these issues.

Order of solutions
2 participants