Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CVXOPTBackend: Set initial problem_name #20332

Closed
mkoeppe opened this issue Mar 31, 2016 · 12 comments
Closed

CVXOPTBackend: Set initial problem_name #20332

mkoeppe opened this issue Mar 31, 2016 · 12 comments
Labels
c: linear programming c: numerical p: major / 3 t: bug
Milestone
sage-7.2

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 31, 2016

CVXOPTBackend forgets to set its initial problem_name, and as a result the MixedIntegerLinearProgram cannot print itself.

sage: p = MixedIntegerLinearProgram(solver="CVXOPT")
sage: p
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
...
TypeError: cannot concatenate 'str' and 'NoneType' objects
<repr(<sage.numerical.mip.MixedIntegerLinearProgram at 0x190970a70>) failed: TypeError: cannot concatenate 'str' and 'NoneType' objects>

CC: @dimpase @ingolfured @videlec @jdemeyer

Component: numerical

Author: Matthias Koeppe

Branch/Commit: f1af4ec

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/20332
@mkoeppe mkoeppe added this to the sage-7.2 milestone Mar 31, 2016
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 31, 2016

Branch: u/mkoeppe/cvxoptbackend__set_initial_problem_name

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 31, 2016

Commit: eb4649c

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 31, 2016

New commits:

eb4649cCVXOPTBackend: Set initial problem_name

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 5, 2016

comment:3

... needs review ...

@dimpase
Copy link
Member

dimpase commented Apr 5, 2016

comment:4

Can we have a doctest showing that this fixes the error in the ticket description?

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 5, 2016

Changed commit from eb4649c to f1af4ec

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 5, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

f1af4ecCVXOPTBackend: Add test for #20232

@dimpase
Copy link
Member

dimpase commented Apr 5, 2016

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Apr 5, 2016

comment:6

ok, good to go.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 5, 2016

comment:7

Thanks for reviewing, Dima!

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 5, 2016

Author: Matthias Koeppe

@vbraun
Copy link
Member

vbraun commented Apr 6, 2016

Changed branch from u/mkoeppe/cvxoptbackend__set_initial_problem_name to f1af4ec

@vbraun vbraun closed this as completed in 347aa41 Apr 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: linear programming c: numerical p: major / 3 t: bug
Projects
None yet
Milestone
sage-7.2
Development

No branches or pull requests
3 participants
@vbraun @dimpase @mkoeppe