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.

Sign up for GitHub

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

Jump to bottom

Fix/hybrid valid #260

Merged

cdiener merged 4 commits into master from fix/hybrid_valid

Apr 24, 2024

Member

cdiener commented Apr 12, 2024

fix the feasibility check in the hybrid solver
description of feature/fix
tests added/passed
add an entry to the next release

This fixes the feasibility check and a small issue in building the problem matrices. This now enables the warm start even across HIGHS/OSQP.

cdiener added 4 commits

April 11, 2024 14:04


          fix check for valid solution

6346f8c


          use right setting for tolerance

5c8f75b


          fix the feasibility check

d9ea79b


          add the hybrid solver to the engine selection

f28ddb7

Midnighter approved these changes

View reviewed changes

Member

Midnighter left a comment

Not a fan of the super short variable names, but otherwise it looks good to me.

cdiener merged commit 8cc7cfd into master

13 checks passed

Midnighter deleted the fix/hybrid_valid branch

April 24, 2024 20:52

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

Labels

None yet