#10530: De Bruijn Sequence construction for combinat
------------------------------------------------+---------------------------
Reporter: eviatarbach | Owner: eviatarbach
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-4.7.1
Component: combinatorics | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Nicolas M. Thiéry, Nathann Cohen | Author: Eviatar Bach
Merged: | Dependencies:
------------------------------------------------+---------------------------
Comment(by ncohen):
Yooooo !
> I tried building the documentation, and I'm getting the same warning. Do
you think it would be okay to submit it anyways?
No, it has to be fixed ! The release manager will set it back to
needs/work otherwise anyway `;-)`
> What's the point of this test?
As I trold you during my latest review, the
{{{
if not True:
}}}
Should be repaced by
{{{
D.an_element() in D
}}}
The aim is to check that all the small DeBruijnSequences that the code
return are indeed valid `:-)`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10530#comment:49>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
