Wrong LP solver ordering

[nathanncohen]

At the moment, GLPK is the solver used regardless of the presence of CBC or CPLEX. This is just because of a line written ten lines too high in the file !

Nathann

Component: numerical

Author: Nathann Cohen, Leonardo Sampaio

Reviewer: Nathann Cohen, Leonardo Sampaio

Merged: sage-4.5.2.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/9570

[nathanncohen]

comment:1

Attachment: trac_9570.patch.gz

[sagetrac-lsampaio]

Fixes the AttributeError that was returned when no mip solver was specified by the user

[sagetrac-lsampaio]

comment:2

Attachment: trac_9570-fix.patch.gz

I applied your patch, but while trying to solve an MIP without specifying a solver, I've got an AttributeError, since the attribute '_default_solver' was not defined.
I just fixed this by adding a line stating that _default_solver = None.
If you agree with my changes, I think the patch can be said to be reviwed.

[nathanncohen]

comment:3

Excellent ! Thank you very much for your help :-)

Both apply fine and in the end, it works... Now the annoying part is #8880 because CPLEX is called by next-to-any method in the LP library XD

Nathann

[nathanncohen]

Author: Nathann Cohen, Leonardo Sampaio

[nathanncohen]

Reviewer: Nathann Cohen, Leonardo Sampaio

[dandrake]

Merged: sage-4.5.2.alpha1

[dandrake]

comment:4

Leonardo -- be sure to use informative commit messages for your patches. "fix" is not very helpful. :) Always include a ticket number, too. I changed your commit message to "ticket 9570: insure _default_solver attribute exists".

both patches merged in 4.5.2.alpha1.

[sagetrac-lsampaio]

comment:5

ok, thanks for the advice =)