Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Palindromes, list of LPS lengths and list of maximal palindromes lengths #18060

Closed
nadialafreniere opened this issue Mar 26, 2015 · 14 comments
Closed

Comments

@nadialafreniere
Copy link
Contributor

This is a new ticket for the algorithm I proposed in ticket #16935. The reason I create a new ticket for it is that there were modifications in the finite_word class (I did not know how to solve that problem otherwise). This aims to integrate the modifications already done by others in the class and the comments that have were written about ticket #16935.

CC: @seblabbe @videlec

Component: combinatorics

Keywords: words, finite words, palindromes

Issue created by migration from https://trac.sagemath.org/ticket/18060

@nadialafreniere
Copy link
Contributor Author

@nadialafreniere
Copy link
Contributor Author

Author: Nadia Lafrenière

@nadialafreniere
Copy link
Contributor Author

@nadialafreniere
Copy link
Contributor Author

Changed keywords from none to words, finite words, palindromes

@nadialafreniere

This comment has been minimized.

@videlec
Copy link
Contributor

videlec commented Mar 26, 2015

comment:4

Bonsoir Nadia,

I do not quite understand why you opened this new ticket. Is it exactly the same purpose as for ticket #16935? Furtheromre, in this ticket there seems to be a problem with the branch field which does not appear in the ticket description.

Vincent

@nadialafreniere
Copy link
Contributor Author

@seblabbe
Copy link
Contributor

Commit: 160ccb0

@seblabbe
Copy link
Contributor

comment:6

This is a duplicate of #16935. Creating a new branch does not need the creation of a new ticket!


New commits:

160ccb0Initial commit

@seblabbe seblabbe removed this from the sage-6.6 milestone Mar 27, 2015
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 8, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

67ac06fMerge branch 'u/nadialafreniere/palindromes...` into t/16935
ef438b5Improving length_maximal_palindrome method
d535b5fModified of lengths_maximal_palindromes to answer previous question, shortened palindromes function

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 8, 2015

Changed commit from 160ccb0 to d535b5f

@videlec
Copy link
Contributor

videlec commented Apr 8, 2015

@videlec
Copy link
Contributor

videlec commented Apr 8, 2015

Changed commit from d535b5f to none

@videlec
Copy link
Contributor

videlec commented Apr 8, 2015

Changed author from Nadia Lafrenière to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants