-
Notifications
You must be signed in to change notification settings - Fork 25
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
Support for MOI 1 #106
Comments
Closed by #110 |
Was that all that had to be done to transition to MOI 1.0, or are there other parts that we don't implement that we still need to? |
Yeah maybe leave this open. There's actually a bunch of issues when I look through the code. These should all be implemented and fixed: Lines 69 to 93 in c2ff58b
Let me take a look through. I have some time, and it'd be good to fix up OSQP.jl properly for JuMP 1.0. |
If you could hold off doing any major updates to OSQP.jl for a while that would be appreciated. We are working on rebasing it to work with the updated API and library components that will be in v1 of the C library (hopefully being released soon), and have started those modifications in the |
Alternately, if you want to target the branch we are working on the 1.0 version on, feel free to make more changes. But we don't have the jll it refers to pushed anywhere right now. |
Okay, sure. |
OSQP currently pins MOI to 0.10.3. A few days ago, MOI as reached 1.0. It would be great to support MOI 1.0 within OSQP and adjust the compat bounds accordingly.
The text was updated successfully, but these errors were encountered: