-
Notifications
You must be signed in to change notification settings - Fork 122
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
HiGHS parameter pruning #668
Conversation
Gotten rid of further non-essential parameters
Re-introduced crossover
@JesseJenkins this PR is ready for review now. I have checked against Gurobi and CPLEX and I have included the bare minimum number of settings in configure_highs.jl to the extent those correspond to similar set of settings parameters in the other two solvers. There are some for which I couldn't find an exact same or similar counterpart. Please let me know if this list looks good, or, if we need to include or get rid of any more parameters. |
@sambuddhac In HiGHS solver settings, I think we can remove also each of these:
Dont we need a MIP gap setting too if someone is using HiGHS for MILP (which is not recommended, but possible)? Restore these?
|
The Cbc default settings also don't look like ones a user would frequently set, are they? And it's missing basic ones like MIP gap setting (and Cbc is the MILP solver). Can you please check configure settings for Cbc as well?
|
Yes, I can reformat the Cbc Also |
Ah ... yeah, forgot that we have room for doing MILP also. Okay, I'll get those in. |
Removed infinite bound, infinite cost, objective bound, objective target, small matrix and large matrix values.
Re-introduced MIP rel gap and MIP abs gap
@JesseJenkins I think now this one should look good? I am creating a separate PR for the Cbc solver. Will pass on to you for reviewing that once finished. |
Description
What type of PR is this? (check all applicable)
Related Tickets & Documents
Checklist
How this can be tested
Post-approval checklist for GenX core developers
After the PR is approved