#17979: Reimplementation of IntegerListsLex
-------------------------------------+-------------------------------------
       Reporter:  aschilling         |        Owner:
           Type:  defect             |       Status:  needs_work
       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                |  0bff4490123fc1366799be4a92d5d5ddfb82a5cc
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by jdemeyer):

 Replying to [comment:439 ncohen]:
 > Now, I agree that this is hard to fix inside of the class. Thus instead
 of
 > claiming that it is not a bug by adding a line of documentation (which
 is why
 > you are rewriting the class today) please consider deprecating it.

 I completely agree with you in principle. However, I would be against
 adding even more changes to other parts of Sage.

 As far as I can tell, within the Sage library, `IntegerListsLex` is only
 called either with a single number or with `NN` as argument. I agree with
 deprecating other arguments (in particular, lists). Deprecating
 `IntegerListsLex(NN, ...)` should be done in a follow-up ticket.

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