-
-
Notifications
You must be signed in to change notification settings - Fork 390
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
testing infrastructure #14
Comments
Basic tests for Variable, AffExpr, and QuadExpr done - see output for a TODO. |
I added some functionality to efficiently merge duplicate coefficients, if you'd like to take a look to do a similar thing for printing. |
Todo: test coverage for 8ede799 -like issues |
I added some of this in 1f3a192. I wouldn't call it complete though. |
Nice - maybe I'm missing it, but does it have the test that caused the bug you fixed recently, the one where the indexing was dependent e.g. i=1:N, j=1:i |
No, it doesn't. |
Just added: cb2d793. |
👍 |
Doing pretty good now. Constraints could probably do with more test coverage, I think a couple of unit tests each for MPS and LP just to ensure we don't accidently introduce bugs would be good. Then I guess whatever random models we come up for those, also run them through solve. Once thats done, I'd be willing to close this issue, with the idea that we be from now on, so a) no pushing without running tests, and b) once we start adding stuff, e.g. quadratic, duals, we update/add tests |
Sounds like a plan. |
Actually, it might be helpful to have TravisCI integration especially if we're testing with different solvers. This way, issues with Clp and CoinMP packages will also show up. Not high priority, but definitely useful. |
Oh yeah, good point. A test for the wider infrastructure! |
Added Travis-CI stuff |
Once you consider constraints, and we add tests, I'd be keen to close this issue. We can add tests for quadratic objective as we go. |
The travis build passes now, and constraints are a separate issue, so let's close this. |
No description provided.
The text was updated successfully, but these errors were encountered: