Skip to content

Conversation

Lazeeez
Copy link
Contributor

@Lazeeez Lazeeez commented Nov 9, 2021

Description of Change

Partition problem is to determine whether a given set can be partitioned into two subsets such that the sum of elements in both subsets is the same.

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes: This implementation has dynamic programming in it with the time complexity of O(sum*n) and space complexity of O(sum*n) where sum is the sum of all the elements of vector

@Lazeeez Lazeeez changed the title feat: Added partition_problem.cpp feat: Added Solution of Partition Problem. Nov 9, 2021
@Panquesito7 Panquesito7 added the enhancement New feature or request label Nov 9, 2021
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Awesome work! 🚀

@Panquesito7 Panquesito7 added the requested changes changes have been requested label Nov 9, 2021
Lazeeez and others added 2 commits November 9, 2021 23:09
Co-authored-by: David Leal <halfpacho@gmail.com>
Co-authored-by: David Leal <halfpacho@gmail.com>
Lazeeez and others added 3 commits November 10, 2021 07:42
Co-authored-by: David Leal <halfpacho@gmail.com>
Co-authored-by: David Leal <halfpacho@gmail.com>
Co-authored-by: David Leal <halfpacho@gmail.com>
@Panquesito7 Panquesito7 changed the title feat: Added Solution of Partition Problem. feat: Added Solution of Partition Problem Nov 10, 2021
Lazeeez and others added 2 commits November 10, 2021 16:49
Co-authored-by: David Leal <halfpacho@gmail.com>
Co-authored-by: David Leal <halfpacho@gmail.com>
@Lazeeez Lazeeez requested a review from Panquesito7 November 10, 2021 11:34
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Awesome work! 🚀 Thanks so much for your contribution! 😄👍

@Panquesito7 Panquesito7 added approved Approved; waiting for merge and removed requested changes changes have been requested labels Nov 10, 2021
@Panquesito7
Copy link
Member

Will merge for now. If any of you (@Amino19, @ayaankhan98, @mishraabhinn, or anyone else) have any objections, please comment here and make a separate PR to fix that. Thank you. 🙂

@Panquesito7 Panquesito7 merged commit 5147306 into TheAlgorithms:master Nov 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approved; waiting for merge enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants