-
Notifications
You must be signed in to change notification settings - Fork 118
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
Fix compatibility of OSQP 1 when using Windows #163
Conversation
The macOS failure are due to #161, and not caused by this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to understand, how did you come up with this solution?
Based on osqp/osqp#576 I understood that we had to stop using So I compare
So, I looked at the logic that was in |
@traversaro Hello, I am having problems writing MPC programs in VS2015 using eigen(v3.4.0),osqp(v0.6.3),osqp-eigen(v0.8,1), |
Can you open a separate issue for this? Having new problems in closed issues is not easy to deal with at all, thanks. |
Yes,i open a separate issue. Problems writing MPC programs using eigen,osqp,osqp-eigen in VS2015 #169 |
Please avoid spamming in other discussions in that case then, especially if you were just waiting for an answer only for a few hours. In general people working on research software need to go through a lot of notifications, so if we are able to minimize unnecessary one that would be great, thanks! |
@traversaro I apologize for this and will take care of it in the future. Can you answer my question? |
If the problem is the same in #169, @S-Dafarra is already helping you there, please avoid unnecessary mentions, as I already explained before, thanks! |
Fix #146, this should reduce new issues being opened in the repo.
fyi @gergondet