#8739: Addition of Kolakoski word
----------------------------------------------+-----------------------------
   Reporter:  abmasse                         |       Owner:  sage-combinat   
       Type:  enhancement                     |      Status:  needs_review    
   Priority:  minor                           |   Milestone:  sage-4.6.1      
  Component:  combinatorics                   |    Keywords:  Kolakoski, words
     Author:  Alexandre Blondin Massé         |    Upstream:  N/A             
   Reviewer:  Nathann Cohen, Sébastien Labbé  |      Merged:                  
Work_issues:                                  |  
----------------------------------------------+-----------------------------
Changes (by newvalueoldvalue):

  * reviewer:  => Nathann Cohen, Sébastien Labbé
  * author:  => Alexandre Blondin Massé


Comment:

 I just added a new patch which applies on the two precedent. All tests
 passed. Documentation builds fine. The Kolakoski word satisfies its
 definition for prefixes up to 100000.

 To me it is a positive review. I let Alexandre change the status of the
 ticket to positive review if he agrees with my changes.

 Maybe Nathann wants to take a look?

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

Reply via email to