-
Notifications
You must be signed in to change notification settings - Fork 40
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 mosek 9 (local branch) #1454
Conversation
…expectations. Need to reformulate the primal problem in order to align with gpkit expectations. This may cause things to break; committing current work so as to not lose it.
…because of more serious unit-conversion or string-printing issues.
…ssed as linear inequalities.
… with MOSEK version >= 9.
…specifically commit 1e67913)
(nb that lint issues seem to have also arisen in pylint-dev/pylint#2436, and so might point at pylint on jenkins being out of date, but I just fixed them with a disable for now) |
Oops, I done messed up. Fixing... |
test this please |
Yay, two green checks. Just macys_VM left in dependencies. Any ideas @bqpd? |
test models please |
I think this has something to do with it
|
@galbramc why only in dependencies though? |
also, Windows10 registered a KeyboardInterrupt??? |
These tests have absolutely stupefied me. |
updated the Dependencies script to match Unit more |
How are things going putting all the testing scripts in that separate repo? That should help make sure they are all consistent. |
test this please |
1 similar comment
test this please |
patience. There are lots of things running right now. We only have so many computers. |
Oh I didn't see that you aborted a bunch of stuff. |
ahaha no worries! just confirming that even with a straight-up duplicate of the script that works on units it doesn't work on macy_vm dependencies |
ahaha no worries! just confirming that even with a straight-up duplicate of the script that works on units it doesn't work on macy_vm dependencies because the path to that executable is too long for the binary. taking mosek off of dependency checking for now. |
test this please |
3 similar comments
test this please |
test this please |
test this please |
test models please |
@bqpd (cc @rileyjmurray @1ozturkbe) This PR seems to break the build pipeline for a linux-base-image docker container we use. I know your build testing includes a linux machine so I'm particularly surprised this is happening, but do you have any ideas why we might be seeing floating point exceptions? No issues building (locally) on OSX.
Let me know what other information would be useful to better understand the issue. |
@pgkirsch This doesn't change the build script much at all, so I wonder if it has more to do with the docker re-build? Could you confirm it works on the commit prior to this PR's merging? |
@pgkirsch @bqpd @1ozturkbe I'd recommend creating a separate GitHub issue for this build problem. You can reference the PR with a hashtag |
Yeah fair enough, good idea @rileyjmurray. |
local version of #1452
This is fantastic, thanks @rileyjmurray! Making a few small changes to pass tests locally & on some of our larger SP models. @1ozturkbe, it seems like it doesn't pass on SPaircraft at present, and fails on robust (as does mosek, though?).