Updates to master #92
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Only update the _variables_to_constriants_mapping dict when new_name != old_name
Only _round_primals_to_bounds if solution is optimal
Default is set to 1e-9
E.g. numpy and sympy
* feat: add get_linear_coefficients method Abstract non-implemented method * feat: implement glpk get_linear_coefficients method * fix: typo * feat: implement get_linear_coefficients in cplex * feat: implement get_linear_coefficients in gurobi * feat: implement get_linear_coefficients in scipy * test: add tests for get_linear_coefficients * test: more tests for get_linear_coefficients * fix: flake8 * fix: missing arguments in test * fix: make scipy consistent with other interfaces the get_linear_coefficients and set_linear_coefficients methods now raise an exception if the objective/constraint doesn’t belong to a model
Some cplex versions don’t have all the statuses that optlang tries to import, which would result in a crash.
For range constraints, rhs is now equal to lb
This is due to a bug where gurobi can get stuck
* checking if __del__ is needed * testing against miplib * using problems from miplib2003 * needs to implement variable comparison for problems that converge * more problems runtime: 10 min per problem * fixed milp test value (same as in devel) * delete problem from memory only if it exists (__del__ method) * add a .keep file to keep the miplib2003 directory on tests/data * miplib test with cplex - some problems also fail... * need to implement a parser for .sol files * set 9 min limit for optimization to run, otherwise travis halts after 10 min without output * refactor: tests and move into slow_tests top directory * style: flake8 * fix: no point in searching ImportError for 'cplex' * test: add slow MILP tests for gurobi
* Fix LB/UB Coefficient Change (#82) * Fix LB/UB Coefficient Change If coeff is negative, the lower bound could become larger than the upper bound causing an error. Now, the upper bound is first removed and then both bounds are reapplied with the shift. * Update interface.py * test: add unittest for constraint expression standardisation * fix: flake8 * fix: flake8 again * refactor: simplify logic in constraint offset handling
* feat: add cloned model tests to netlib test suites * chore: add cloned model tests also for gurobi * fix: critical bug affecting ranged constraints in cplex interface * test: skip netlib cloning tests on travis
* Various fixes (#78) fix: objective.value should return None if not in model fix: get correct primal values from glpk MIPs test: add test for obj value without model fix: allow older versions of cplex Numerous gurobi fixes Miplib test suite * Add Gitter badge * Fix gitter badge location * Fix gitter badge 2
Codecov Report
@@ Coverage Diff @@
## master #92 +/- ##
========================================
+ Coverage 88.7% 88.8% +0.1%
========================================
Files 8 8
Lines 2381 2385 +4
Branches 542 542
========================================
+ Hits 2112 2118 +6
+ Misses 167 166 -1
+ Partials 102 101 -1
Continue to review full report at Codecov.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix: make sure that sloppy arg is being passed to super (#83)
Netlib tests test also cloned models (#79)
Add a Gitter chat badge to README.rst (#86) (#88)