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

String based Algorithms added #654

Merged
merged 17 commits into from
Oct 14, 2023
Merged

String based Algorithms added #654

merged 17 commits into from
Oct 14, 2023

Conversation

architO21
Copy link
Contributor

@architO21 architO21 commented Oct 5, 2023

Proposed changes

I have added a new directory known as String based algorithms and added string matching codes inside of it

Types of changes

What types of changes does your content introduce to this project?
Put an x in the boxes that apply (without "`")

  • Proofreading (fix spelling, grammar, punctuation, accuracy) Proofreading
  • [ x] New content (new section, new readme, etc.)
  • [ x] Modifications (modifying existing content)
  • Other (please describe)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. Don't hesitate to ask if you're unsure about any of them. We're here to help! This simply reminds us of what we will look for before merging your code.
Note: If you submit code make sure to test it

  • [ x] Tested any code that was added/modified in this PR
  • Added external links
  • Added internal links
  • Added images
  • Added videos
  • Added code snippets

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Adding your name to the contributor's list

Comment "@all-contributors please add @your_username for content" to add your name to the contributor's list.
You can change the "content" with any of this

Reviewers:

  • @username

Please ensure that you are not adding any sensitive information like API keys, secrets, passwords, tokens, usernames, or other credentials. Refrain from adding your name, email, or other personal information.

I have made a new directory known as string based algorithms in aglorithms directory and have added KMP algorithm inside of it
In this commit I have added RabinKarp algorithm for string matching in the StringbasedAlgorithms directory
@the-label-bot the-label-bot bot added kind/documentation kind/documentation size/L size/L labels Oct 5, 2023
@the-label-bot
Copy link

the-label-bot bot commented Oct 5, 2023

The Label Bot has predicted the following:

Category Value Confidence Applied Label
Kind documentation 0.882 ✔️
Size L 0.933 ✔️

Copy link
Owner

@shhossain shhossain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please ensure that your code is thoroughly documented with meaningful comments and includes comprehensive explanations in the algorithms section of the README file.

Added comments to KMP and explained the algorithm via an example
Added a new section in the readme for string based algorithms.
Added a readme file for KMP explained it using an example and also added the code for implementing it in the readme
Added a a readme version for Rabin Karp algorithm and ran it on system tests
added a new readme file in the directory string based algorithms all new code snippets and algorithms can now be added here making it easier and less complex than merging all in one
@architO21
Copy link
Contributor Author

Please ensure that your code is thoroughly documented with meaningful comments and includes comprehensive explanations in the algorithms section of the README file.

I have applied the changes that you had mentioned in my code and also updated the readme file

Copy link
Owner

@shhossain shhossain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change this:

  1. Fix the code format Tutorial
  2. Rename the folder from Stringbasedalgorithms to String Based Algorithms.

Finally, Thank you for your valuable contribution.

@the-label-bot the-label-bot bot added size/XS size/XS and removed size/L size/L labels Oct 13, 2023
I have added a readme file that contains code for one of the most important string based searching algorithms which is KMP
@the-label-bot the-label-bot bot added size/L size/L and removed size/XS size/XS labels Oct 13, 2023
Created a readme file where I have provided in depth explanation for Rabin Karp algorithm for string based pattern searching
@architO21
Copy link
Contributor Author

architO21 commented Oct 13, 2023

I have changed the name of the folder and have written code in the format as you had mentioned. Please review my changes

Copy link
Owner

@shhossain shhossain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work.

@shhossain shhossain merged commit f8974c0 into shhossain:main Oct 14, 2023
1 check failed
@shhossain shhossain added the hacktoberfest-accepted hacktoberfest-accepted label Oct 14, 2023
@shhossain
Copy link
Owner

@all-contributors add @architO21 for code and content.

@allcontributors
Copy link
Contributor

@shhossain

I've put up a pull request to add @architO21! 🎉

@architO21
Copy link
Contributor Author

Thankyou @shhossain this was my first contribution in open source, I got to learn a lot working with you

@shhossain
Copy link
Owner

Congratulations on your first open-source contribution! I'm glad you had a great learning experience. Keep it up! @architO21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted hacktoberfest-accepted kind/documentation kind/documentation size/L size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants