Skip to content

Support early solver stop without solution in HiGHS#114

Merged
lovasoa merged 1 commit intorust-or:mainfrom
TeXitoi:support_early_stop_without_solution
Nov 12, 2025
Merged

Support early solver stop without solution in HiGHS#114
lovasoa merged 1 commit intorust-or:mainfrom
TeXitoi:support_early_stop_without_solution

Conversation

@TeXitoi
Copy link
Contributor

@TeXitoi TeXitoi commented Nov 10, 2025

If you put a stopping option as .set_option("mip_max_nodes", 10), the solver might stop without finding a feasible solution. This modification handles this case by checking that a feasible solution is available, returning Err(ResolutionError::Other("NoSolutionFound")) if not.

depends on rust-or/highs#32

@TeXitoi TeXitoi force-pushed the support_early_stop_without_solution branch 3 times, most recently from 24d2f04 to aaa2315 Compare November 10, 2025 16:27
@lovasoa lovasoa marked this pull request as draft November 10, 2025 20:50
@lovasoa
Copy link
Collaborator

lovasoa commented Nov 10, 2025

Thanks !

I switched the pr to draft, you can mark it as "ready for review" when the upstream changes are merged

@TeXitoi TeXitoi force-pushed the support_early_stop_without_solution branch from aaa2315 to dc56a85 Compare November 12, 2025 09:40
@TeXitoi TeXitoi marked this pull request as ready for review November 12, 2025 09:41
Copy link
Collaborator

@lovasoa lovasoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good for me, thanks @TeXitoi

If you put a stopping option as `.set_option("mip_max_nodes", 10)`, the solver
might stop without finding a feasible solution. This modification handles this
case by checking that a feasible solution is available, returning
`Err(ResolutionError::Other("NoSolutionFound"))` if not.

depends on rust-or/highs#32
@TeXitoi TeXitoi force-pushed the support_early_stop_without_solution branch from f7a219f to a3a1110 Compare November 12, 2025 09:49
@lovasoa lovasoa merged commit f10e1e8 into rust-or:main Nov 12, 2025
12 checks passed
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

Successfully merging this pull request may close these issues.

2 participants