#17979: Reimplementation of IntegerListsLex
-------------------------------------+-------------------------------------
Reporter: aschilling | Owner:
Type: defect | Status: needs_review
Priority: blocker | Milestone: sage-6.6
Component: combinatorics | Resolution:
Keywords: days64 | Merged in:
Authors: Bryan Gillespie, | Reviewers: Nathann Cohen, Jeroen
Anne Schilling, Nicolas M. Thiery | Demeyer, Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/ticket/17979 | 80ad5c8897ea2714a76a7a341b5652d622ae291a
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:316 jdemeyer]:
> Design comment: the code for `Envelope` has a lot of duplication because
it needs to deal with upper and lower bounds. Instead, I would propose to
have `Envelope` deal ''only'' with upper bounds. The `Envelope` for the
lower bound could then be implemented by adding minus signs in the input
and output of `Envelope`.
I am also not so happy about this. I had considered doing something
along the lines you propose, and ended up dropping it for now. This is
mainly because, in the next step, the main functionality to add will
be the computation of the area below the envelope for which we won't
have this symmetry anymore. Also, at this point, we are speaking about
5-6 lines of code that are duplicated very locally. Doing the magic
would not take much less than this, and the code would be less
straightforward. Sooo, well.
--
Ticket URL: <http://trac.sagemath.org/ticket/17979#comment:354>
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.
