#16504: A mandatory 'nonnegative' argument for
MixedIntegerLinearProgram.new_variable() until the standard changes
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.3
      Component:  linear             |   Resolution:
  programming                        |    Merged in:
       Keywords:                     |    Reviewers:  Jeroen Demeyer
        Authors:  Nathann Cohen      |  Work issues:  nonnegative=False
Report Upstream:  N/A                |  doctests, avoid isinstance()
         Branch:  u/ncohen/16504     |       Commit:
   Dependencies:                     |  b1f43af754d4ab7e136015519c7fc0cdbe32dd36
                                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by ncohen):

 > Am I correct that this is a safe change in the sense that the existing
 code using `nonnegative=True` as default would either continue to work, or
 will refuse to work? I.e. people won't possibly start getting wrong
 answers?

 It will work, and there should be no wrong answer (unless there is a bug
 somewhere). The default behaviour is now "nonnegative + warning", and the
 correct way to use this class is to specify a value for nonnegative.

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/16504#comment:31>
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.

Reply via email to