Skip to content

Added bounds to LCP calculation. Added tests for LCP array and more sample output.#218

Open
killacoug9 wants to merge 1 commit intoindy256:mainfrom
killacoug9:lcp-calculation-bug
Open

Added bounds to LCP calculation. Added tests for LCP array and more sample output.#218
killacoug9 wants to merge 1 commit intoindy256:mainfrom
killacoug9:lcp-calculation-bug

Conversation

@killacoug9
Copy link
Copy Markdown

fixes issue #180

@killacoug9
Copy link
Copy Markdown
Author

Added bounds to LCP to fix issue . Added tests for LCP array and more sample output.
example of current output:
"""
Test 1 - Expected: [], Got: 0 0 0
Test 2 - Expected: [2, 0, 1], Got: 2 0 1
Test 3 - Expected: [3, 2, 1], Got: 1 2 3
Test 4 - Expected: [], Got: 0 0 0 0 0
Test 6 - Expected: [], Got:

Example test for 'abcab'
3 0 4 1 2
2 0 1 0

Example test for 'banana'
5 3 1 0 4 2
1 3 0 0 2

1678.3 ms
"""

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant