Add missing tests for integer control parameters #12
Conversation
Including: sr_heur, fp_heur, ps_heur, ps_tim_lim, cov_cuts, clq_cuts, mip_gap, presolve, binarize.
There was a problem hiding this comment.
Thanks for adding these tests. It seems like what is being tested is whether or not an error is raised when different parameters are toggled. Is there any way to check the state of the LPX object after calling integer() to test if the control parameter had the intended effect?
| def testSimpleRoundingHeuristic(self): | ||
| """Test the sr_heur parameter.""" | ||
| if env.version<(4,57): return | ||
| legals=True, False |
There was a problem hiding this comment.
Looking at 'src/lp.c', the argument sr_heur is declared to be an integer and not a boolean. Should 0, 1 be included in the list of legal values? Is there anything that can be checked after calling integer() with this argument to check if it was actually used?
That's a good point - I'll look into it. |
I implemented tests for the LP_solver_integer control parameters I added in #10.