Implemented Suffix Tree Data Structure #11554

Ramy-Badr-Ahmed · 2024-09-07T19:25:59Z

Describe your change:

This pull request adds an implementation of the Suffix Tree data structure along with related functionalities, such as pattern searches.

Definition: The Suffix Tree is a compressed trie of all the suffixes of a given text used for for substring search and various other string-processing operations. (URL: Suffix Tree).

Implementation Details:

suffix_tree_node.py: Defines the SuffixTreeNode class, representing nodes in the suffix tree.
build_suffix_tree.py: Builds a suffix tree from the given text and apply pattern searche for a pattern in the built suffix tree.

Example Usage:

example_usage.py: Demonstrates how to use the Suffix Tree implementation with a predefined text and searches for various patterns.

Unit Tests:

test_suffix_tree.py: Validates the correctness of the Suffix Tree implementation, including tests for building the tree and searching for patterns.

Reference

Data Structures and Algorithms in C++, 2nd Edition

for more information, see https://pre-commit.ci

…d_tree/example_usage.py

…d_tree/hypercube_points.py

for more information, see https://pre-commit.ci

…into feature/kd-tree-implementation # Conflicts: # data_structures/kd_tree/nearest_neighbour_search.py

for more information, see https://pre-commit.ci

…ch.py

Co-authored-by: Christian Clauss <[email protected]>

…dtree() to include various checks.

for more information, see https://pre-commit.ci

…into feature/kd-tree-implementation # Conflicts: # data_structures/kd_tree/tests/test_kdtree.py

for more information, see https://pre-commit.ci

…on' into feature/suffix-tree-implementation # Conflicts: # data_structures/suffix_tree/tests/test_suffix_tree.py

for more information, see https://pre-commit.ci

Ramy-Badr-Ahmed

Done. All checks have passed. 👍🏻

Hi, @cclauss & @imSanko: Looking forward for your review 👋

imSanko

Looks fine to me, wait for the merge!

Added some comments to my files in TheAlgorithms#11532, TheAlgorithms#11554.

Ramy-Badr-Ahmed

Added Doc blocks to the rest of my files requesting mentioning me for bugs/corrections.
All checks have passed 👍

* Implemented KD-Tree Data Structure * Implemented KD-Tree Data Structure. updated DIRECTORY.md. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Create __init__.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Replaced legacy `np.random.rand` call with `np.random.Generator` in kd_tree/example_usage.py * Replaced legacy `np.random.rand` call with `np.random.Generator` in kd_tree/hypercube_points.py * added typehints and docstrings * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * docstring for search() * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Added tests. Updated docstrings/typehints * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * updated tests and used | for type annotations * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * E501 for build_kdtree.py, hypercube_points.py, nearest_neighbour_search.py * I001 for example_usage.py and test_kdtree.py * I001 for example_usage.py and test_kdtree.py * Update data_structures/kd_tree/build_kdtree.py Co-authored-by: Christian Clauss <[email protected]> * Update data_structures/kd_tree/example/hypercube_points.py Co-authored-by: Christian Clauss <[email protected]> * Update data_structures/kd_tree/example/hypercube_points.py Co-authored-by: Christian Clauss <[email protected]> * Added new test cases requested in Review. Refactored the test_build_kdtree() to include various checks. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considered ruff errors * Considered ruff errors * Apply suggestions from code review * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update kd_node.py * imported annotations from __future__ * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Implementation of the suffix tree data structure * Adding data to DIRECTORY.md * Minor file renaming * minor correction * renaming in DIRECTORY.md * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considering ruff part-1 * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considering ruff part-2 * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considering ruff part-3 * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considering ruff part-4 * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Considering ruff part-5 * Implemented Suffix Tree Data Structure. Added some comments to my files in TheAlgorithms#11532, TheAlgorithms#11554. * updating DIRECTORY.md * Implemented Suffix Tree Data Structure. Added some comments to my files in TheAlgorithms#11532, TheAlgorithms#11554. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Christian Clauss <[email protected]> Co-authored-by: Ramy-Badr-Ahmed <[email protected]>

Ramy-Badr-Ahmed and others added 30 commits August 28, 2024 13:09

Implemented KD-Tree Data Structure

0d6985c

Implemented KD-Tree Data Structure. updated DIRECTORY.md.

6665d23

[pre-commit.ci] auto fixes from pre-commit.com hooks

6b3d47e

for more information, see https://pre-commit.ci

Create __init__.py

4203cda

[pre-commit.ci] auto fixes from pre-commit.com hooks

3222bd3

for more information, see https://pre-commit.ci

Replaced legacy np.random.rand call with np.random.Generator in k…

a41ae5b

…d_tree/example_usage.py

Replaced legacy np.random.rand call with np.random.Generator in k…

1668d73

…d_tree/hypercube_points.py

added typehints and docstrings

81d6917

[pre-commit.ci] auto fixes from pre-commit.com hooks

6cddcbd

for more information, see https://pre-commit.ci

docstring for search()

8b238d1

Merge remote-tracking branch 'origin/feature/kd-tree-implementation' …

cd1dd9f

…into feature/kd-tree-implementation # Conflicts: # data_structures/kd_tree/nearest_neighbour_search.py

[pre-commit.ci] auto fixes from pre-commit.com hooks

ead2838

for more information, see https://pre-commit.ci

Added tests. Updated docstrings/typehints

543584c

[pre-commit.ci] auto fixes from pre-commit.com hooks

ad31f83

for more information, see https://pre-commit.ci

updated tests and used | for type annotations

1322921

[pre-commit.ci] auto fixes from pre-commit.com hooks

4608a9f

for more information, see https://pre-commit.ci

E501 for build_kdtree.py, hypercube_points.py, nearest_neighbour_sear…

7c1aa7e

…ch.py

I001 for example_usage.py and test_kdtree.py

ba24e75

I001 for example_usage.py and test_kdtree.py

05975a3

Update data_structures/kd_tree/build_kdtree.py

31782d1

Co-authored-by: Christian Clauss <[email protected]>

Update data_structures/kd_tree/example/hypercube_points.py

6a9b3e1

Co-authored-by: Christian Clauss <[email protected]>

Update data_structures/kd_tree/example/hypercube_points.py

2fd24d4

Co-authored-by: Christian Clauss <[email protected]>

Added new test cases requested in Review. Refactored the test_build_k…

2cf9d92

…dtree() to include various checks.

[pre-commit.ci] auto fixes from pre-commit.com hooks

a3803ee

for more information, see https://pre-commit.ci

Considered ruff errors

f1f5862

Merge remote-tracking branch 'origin/feature/kd-tree-implementation' …

ec6559d

…into feature/kd-tree-implementation # Conflicts: # data_structures/kd_tree/tests/test_kdtree.py

Considered ruff errors

5c07a1a

Apply suggestions from code review

3c09ac1

[pre-commit.ci] auto fixes from pre-commit.com hooks

bab43e7

for more information, see https://pre-commit.ci

Update kd_node.py

a10ff15

pre-commit-ci bot and others added 9 commits September 7, 2024 20:32

[pre-commit.ci] auto fixes from pre-commit.com hooks

bac6962

for more information, see https://pre-commit.ci

Considering ruff part-2

87909da

Merge remote-tracking branch 'origin/feature/suffix-tree-implementati…

ce8e35d

…on' into feature/suffix-tree-implementation # Conflicts: # data_structures/suffix_tree/tests/test_suffix_tree.py

[pre-commit.ci] auto fixes from pre-commit.com hooks

288a4f0

for more information, see https://pre-commit.ci

Considering ruff part-3

93e910d

[pre-commit.ci] auto fixes from pre-commit.com hooks

e375de7

for more information, see https://pre-commit.ci

Considering ruff part-4

52ba6d2

[pre-commit.ci] auto fixes from pre-commit.com hooks

531252e

for more information, see https://pre-commit.ci

Considering ruff part-5

7479d2d

algorithms-keeper bot removed the tests are failing Do not merge until tests pass label Sep 7, 2024

Ramy-Badr-Ahmed commented Sep 7, 2024

View reviewed changes

Merge branch 'feature/suffix-tree-implementation'

819f3d1

Ramy-Badr-Ahmed mentioned this pull request Sep 8, 2024

Suffix Tree Data Structure Implementation #11555

Closed

imSanko approved these changes Sep 8, 2024

View reviewed changes

This comment was marked as outdated.

Sign in to view

Ramy-Badr-Ahmed changed the title ~~Suffix Tree Data Structure Implementation~~ Implemented Suffix Tree Data Structure Sep 25, 2024

This comment was marked as outdated.

Sign in to view

Ramy-Badr-Ahmed and others added 6 commits September 28, 2024 14:53

Implemented Suffix Tree Data Structure.

d8eb924

Added some comments to my files in TheAlgorithms#11532, TheAlgorithms#11554.

Merge branch 'feature/suffix-tree-implementation'

c70b3ce

updating DIRECTORY.md

666bb6f

Implemented Suffix Tree Data Structure.

3a6c8a3

Added some comments to my files in TheAlgorithms#11532, TheAlgorithms#11554.

Merge branch 'feature/suffix-tree-implementation'

cd8be17

Merge branch 'TheAlgorithms:master' into master

3de5ff3

Ramy-Badr-Ahmed force-pushed the feature/suffix-tree-implementation branch from ad3f2f6 to 3de5ff3 Compare September 28, 2024 13:10

Ramy-Badr-Ahmed commented Sep 28, 2024

View reviewed changes

Ramy-Badr-Ahmed requested a review from imSanko September 28, 2024 13:14

cclauss approved these changes Sep 28, 2024

View reviewed changes

cclauss merged commit 976e385 into TheAlgorithms:master Sep 28, 2024
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Implemented Suffix Tree Data Structure #11554

Implemented Suffix Tree Data Structure #11554

Ramy-Badr-Ahmed commented Sep 7, 2024 •

edited

Loading

Ramy-Badr-Ahmed left a comment

imSanko left a comment

This comment was marked as outdated.

This comment was marked as outdated.

Ramy-Badr-Ahmed left a comment •

edited

Loading

Implemented Suffix Tree Data Structure #11554

Implemented Suffix Tree Data Structure #11554

Conversation

Ramy-Badr-Ahmed commented Sep 7, 2024 • edited Loading

Describe your change:

Ramy-Badr-Ahmed left a comment

Choose a reason for hiding this comment

imSanko left a comment

Choose a reason for hiding this comment

This comment was marked as outdated.

This comment was marked as outdated.

Ramy-Badr-Ahmed left a comment • edited Loading

Choose a reason for hiding this comment

Ramy-Badr-Ahmed commented Sep 7, 2024 •

edited

Loading

Ramy-Badr-Ahmed left a comment •

edited

Loading