#17320: Memory leaks with LP Solvers are back
-------------------------------------+-------------------------------------
Reporter: pmueller | Owner:
Type: defect | Status: positive_review
Priority: major | Milestone: sage-6.4
Component: linear | Resolution:
programming | Merged in:
Keywords: | Reviewers: Nils Bruin, Nathann
Authors: Nils Bruin, | Cohen
Nathann Cohen | Work issues:
Report Upstream: N/A | Commit:
Branch: public/17320 | 5480f4e85c1a0ff96bd1502c78992073ccb854e5
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by SimonKing):
Replying to [comment:27 pmueller]:
> > Why not? Did you perhaps not import the error first?
> Indeed, I didn't know that one has to import this error. Hmm, still
doesn't look user-friendly to me that the class MixedIntegerLinearProgram
is known to Sage without an import, while the exception belonging to it is
not. -- Peter Mueller
I'd say that having `MixedIntegerLinearProgram` in the global name space
is not good, since most Sage users wouldn't use it. So, it is friendly to
*some* users that it does not need to import it before using.
And `MIPSolverException` is a technicality that is only relevant to
developers or expert users, and thus clearly shouldn't pollute the global
name space.
--
Ticket URL: <http://trac.sagemath.org/ticket/17320#comment:28>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
