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

Time limit not compliant with MOI interface #190

Closed
hlefebvr opened this issue Oct 29, 2023 · 3 comments
Closed

Time limit not compliant with MOI interface #190

hlefebvr opened this issue Oct 29, 2023 · 3 comments

Comments

@hlefebvr
Copy link

hlefebvr commented Oct 29, 2023

Hello,

The HiGHS solver currently compares the time limit with the global run time, i.e., the cumulative run time of all runs with a given HiGHS object.

This is to be changed in version 2.0 as indicated in this HiGHS issue.

In the meantime, this makes HiGHS.jl violating the MOI interface specifications and preventing interchangeability of solvers within the JuMP environment.

An easy fix (until version 2.0 release) is to set the time limit to "getRunTime() + time_limit" where "time_limit" is the actual time limit, instead of "time_limit".

Thank you.

@odow
Copy link
Member

odow commented Oct 29, 2023

This is fixed with HiGHS v1.6:

HiGHS.jl/src/MOI_wrapper.jl

Lines 1817 to 1820 in fa6247b

if Highs_versionMajor() == 1 && Highs_versionMinor() >= 6
# Not available in earlier versions
ret = Highs_zeroAllClocks(model)
_check_ret(ret)

@odow
Copy link
Member

odow commented Oct 29, 2023

See also #125, #162

@hlefebvr
Copy link
Author

Thanks a lot!

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

No branches or pull requests

2 participants