#20354: PPLBackend: Add support for integer variables
-------------------------------------+-------------------------------------
Reporter: mkoeppe | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-7.2
Component: numerical | Resolution:
Keywords: lp | Merged in:
Authors: Matthias Koeppe | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/mkoeppe/pplbackend__add_support_for_integer_variables|
144a970ae97b209acccf6d38de540401c8c8a587
Dependencies: #20351,#20303 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by dimpase):
Replying to [comment:6 mkoeppe]:
> Replying to [comment:5 dimpase]:
> > hmm:
> > {{{
> > cpdef int add_variable(self, lower_bound=0, upper_bound=None,
binary=False, continuous=False, integer=False, obj=0, name=None) except
-1:
> > }}}
> > should we really pick a default (`continuous` seems to be the most
natural) rather than being it total denial?
>
> This idiosyncratic interface (including the documentation that says that
the default value of 'continuous' is True) matches that of the other
backends.
but this is a line from your update, why do you set `continuous=False`?
Something you changed while developing and then forgot to revert?
--
Ticket URL: <http://trac.sagemath.org/ticket/20354#comment:7>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
