| 1 | Right now, the followinjg is happening |

| 2 | {{{ |

| 3 | #!python |

| 4 | sage: g = graphs.PetersenGraph() |

| 5 | sage: g.dominating_set() |

| 6 | IBM ILOG License Manager: "IBM ILOG Optimization Suite for Academic Initiative" is accessing CPLEX 12 with option(s): "e m b q ". |

| 7 | [4, 6, 7] |

| 8 | sage: g.dominating_set(solver = "GLPK") |

| 9 | --------------------------------------------------------------------------- |

| 10 | ValueError Traceback (most recent call last) |

| 11 | |

| 12 | /auto/sop-nas2a/u/sop-nas2a/vol/home_mascotte/ncohen/<ipython console> in <module>() |

| 13 | |

| 14 | /home/ncohen/sage/local/lib/python2.6/site-packages/sage/graphs/generic_graph.pyc in dominating_set(self, independent, value_only, solver, verbose) |

| 15 | 5666 return p.solve(objective_only=True, solver=solver, log=verbose) |

| 16 | 5667 else: |

| 17 | -> 5668 p.solve(solver=solver, log=verbose) |

| 18 | 5669 b=p.get_values(b) |

| 19 | 5670 return [v for v in g.vertices() if b[v]==1] |

| 20 | |

| 21 | /home/ncohen/sage/local/lib/python2.6/site-packages/sage/numerical/mip.so in sage.numerical.mip.MixedIntegerLinearProgram.solve (sage/numerical/mip.c:5813)() |

| 22 | |

| 23 | ValueError: Solver argument deprecated. This parameter now has to be set when calling the class' constructor |

| 24 | }}} |

| 25 | |

| 26 | The new interface between Sage and the LP solvers requires the solver's name to be given when the constructor is called. |

| 27 | |

| 28 | This patch applied, all is fine : |

| 29 | |

| 30 | {{{ |

| 31 | #!python |

| 32 | sage: g = graphs.PetersenGraph() |

| 33 | sage: g.dominating_set() |

| 34 | IBM ILOG License Manager: "IBM ILOG Optimization Suite for Academic Initiative" is accessing CPLEX 12 with option(s): "e m b q ". |

| 35 | [4, 6, 7] |

| 36 | sage: g.dominating_set(solver = "GLPK") |

| 37 | [0, 2, 6] |

| 38 | }}} |

| 39 | |

| 40 | Apply first : |

| 41 | |

| 42 | * #10043 |

| 43 | * #10101 |

| 44 | * #9698 |

| 45 | * #9581 |

| 46 | * #10150 |

| 47 | |

| 48 | This long list of undeserved dependencies is here to prevent this patch from having to be rebased, as it touches many files almost everywhere. |