#19494: Make finite word method nb_subword_occurences_in much faster
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  slabbe                 |       Status:  needs_work
           Type:         |    Milestone:  sage-6.10
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:  Vincent Delecroix
  combinatorics          |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  65dae48833cb03071ac122b6e6bdb6ccf4b0340c
  Sébastien Labbé        |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  public/19494           |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by slabbe):

 5 quick comments:

  - I suggest to replace `'vincent'` by `'vector'`.
  - I suggest to change the default argument to `'vector'`.
  - The doctest testing all algorithms should test all algorithms.
  - How many of the four algorithms do we want to keep?
  - If only one, then none of the above comments applies.

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