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

TR strategy: properly reset the bounds duals and set the warm-start information #83

Merged
merged 6 commits into from
Nov 13, 2024

Conversation

cvanaret
Copy link
Owner

@cvanaret cvanaret commented Nov 11, 2024

Properly update the WarmstartInformation object: handle phase switches, errors, etc.
Two fixes in TrustRegionStrategy:

  • loop over all variables to reset the multipliers when TR constraints are active;
  • reset the feasibility multipliers as well;

@cvanaret cvanaret changed the title TR strategy: properly reset the bounds duals when trust-region constraints are active TR strategy: properly reset the bounds duals and set the warm-start information Nov 11, 2024
@odow
Copy link
Contributor

odow commented Nov 12, 2024

Can we test the TR without BQPD?

@cvanaret
Copy link
Owner Author

Yes, we can assemble a trust-region SLP method with HiGHS as LP solver.
So far, I've fixed lack of convergence on infeasible problems for SLP methods. I'm still working on properly setting the warmstart information.

@leyffer
Copy link
Collaborator

leyffer commented Nov 12, 2024 via email

@odow
Copy link
Contributor

odow commented Nov 12, 2024

It would be good to also test it with BQPD

Sure. But this requires us to be able to compile a version of Uno that includes BQPD on a public CI instance.

@cvanaret cvanaret merged commit ecbb89a into main Nov 13, 2024
3 checks passed
@cvanaret cvanaret deleted the TR_duals_fix branch November 13, 2024 15:08
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.

3 participants