#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
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/ticket/17979 | 3356cc375a40ddbff4efc8d1e76512ba027a25de
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:191 jdemeyer]:
> This hangs:
> {{{
> sage: list(IntegerListsLex(1, min_length=2, min_slope=0, max_slope=0))
> }}}
> (it works fine without the `min_length` though)
You really are shaking this guy out! That's good :-)
We looked this up with Anne and the hang was in `._possible_m`; the issue
was similar to one we had elsewhere: if the ceiling is 0 at some point and
max_slope=0, then we know this should be treated as if there was a ceiling
limit of 0. In fact, this was already tested, but not on a moving
position; so this was a 1 character fix at the end.
I'll document this method in detail later today.
--
Ticket URL: <http://trac.sagemath.org/ticket/17979#comment:218>
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.
