#14969: Longest common subword
---------------------------------+----------------------------
Reporter: ncohen | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.12
Component: combinatorics | Resolution:
Keywords: | Merged in:
Authors: Nathann Cohen | Reviewers: Hugh Thomas
Report Upstream: N/A | Work issues:
Branch: | Dependencies:
Stopgaps: |
---------------------------------+----------------------------
Changes (by hthomas):
* status: needs_review => needs_work
* reviewer: => Hugh Thomas
Comment:
Hi Nathann!
Review patch uploaded. I don't think it will be controversial. I removed
one line of code which did nothing useful. (Please confirm.)
The commit message on the original patch should be, um, more descriptive.
Other than that, I am ready to give it a positive review if you approve my
changes.
What a nice algorithm!
The same approach could be used to find all that longest common subwords.
Do you think that would be useful? To me it seems at least as natural.
cheers,
Hugh
--
Ticket URL: <http://trac.sagemath.org/ticket/14969#comment:2>
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/groups/opt_out.
