[Matrix] Max area of island problem solved DFS algorithm by kondekarshubham123 · Pull Request #6918 · TheAlgorithms/Python · GitHub
Skip to content

[Matrix] Max area of island problem solved DFS algorithm - #6918

Merged
cclauss merged 25 commits into
TheAlgorithms:masterfrom
kondekarshubham123:matrix/max_area_of_island
Oct 18, 2022
Merged

[Matrix] Max area of island problem solved DFS algorithm#6918
cclauss merged 25 commits into
TheAlgorithms:masterfrom
kondekarshubham123:matrix/max_area_of_island

Conversation

@kondekarshubham123

@kondekarshubham123 kondekarshubham123 commented Oct 10, 2022

Copy link
Copy Markdown
Contributor

Describe your change:

Maximum area of island problem added.

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.
  • All function parameters and return values are annotated with Python type hints.
  • All functions have doctests that pass the automated testing.
  • All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
  • If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.

@algorithms-keeper algorithms-keeper Bot added the awaiting reviews This PR is ready to be reviewed label Oct 10, 2022
CaedenPH
CaedenPH reviewed Oct 10, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py
Comment thread matrix/max_area_of_island.py Outdated
kondekarshubham123 and others added 3 commits October 10, 2022 23:01
@kondekarshubham123 @CaedenPH
Update matrix/max_area_of_island.py
6cbfc10 
Co-authored-by: Caeden <caedenperelliharris@gmail.com>
@kondekarshubham123 @CaedenPH
Update matrix/max_area_of_island.py
8e34544 
Co-authored-by: Caeden <caedenperelliharris@gmail.com>
@kondekarshubham123 @CaedenPH
Update matrix/max_area_of_island.py
08decd3 
Co-authored-by: Caeden <caedenperelliharris@gmail.com>
@algorithms-keeper algorithms-keeper Bot added tests are failing Do not merge until tests pass and removed tests are failing Do not merge until tests pass labels Oct 10, 2022
@kondekarshubham123

kondekarshubham123 commented Oct 10, 2022

Copy link
Copy Markdown
Contributor Author

kothekarakshata
kothekarakshata approved these changes Oct 10, 2022
View reviewed changes

@kothekarakshata kothekarakshata left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good, new question

kothekarakshata
kothekarakshata approved these changes Oct 10, 2022
View reviewed changes

@kothekarakshata kothekarakshata left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

viewed, good work

@kondekarshubham123 kondekarshubham123 changed the title Max area of island problem solved DFS algorithm; a pr under Hacktoberfest Max area of island problem solved DFS algorithm Oct 11, 2022
@kondekarshubham123 kondekarshubham123 changed the title Max area of island problem solved DFS algorithm [Matrix] Max area of island problem solved DFS algorithm Oct 11, 2022
@kondekarshubham123 kondekarshubham123 mentioned this pull request Oct 11, 2022
Closed
1 task
CenTdemeern1
CenTdemeern1 suggested changes Oct 11, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py
@kondekarshubham123
kondekarshubham123 requested review from CaedenPH and CenTdemeern1 and removed request for CaedenPH and CenTdemeern1 October 11, 2022 11:10
@algorithms-keeper algorithms-keeper Bot added the tests are failing Do not merge until tests pass label Oct 11, 2022
@kondekarshubham123
kondekarshubham123 requested review from CaedenPH and removed request for CenTdemeern1 October 11, 2022 11:13
CenTdemeern1
CenTdemeern1 reviewed Oct 11, 2022
View reviewed changes

@CenTdemeern1 CenTdemeern1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would recommend making some changes here, but I suppose these aren't explicitly required...

Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py Outdated
@kondekarshubham123

kondekarshubham123 commented Oct 12, 2022

Copy link
Copy Markdown
Contributor Author

yes @CenTdemeern1 , I had put : set back
Thanks

@kondekarshubham123

kondekarshubham123 commented Oct 17, 2022

Copy link
Copy Markdown
Contributor Author

Hi @cclauss or @dhruvmanila,
Please review this PR as well

Thanks

CaedenPH
CaedenPH reviewed Oct 17, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py Outdated
Comment thread matrix/max_area_of_island.py
kondekarshubham123 and others added 3 commits October 18, 2022 00:31
@kondekarshubham123 @CaedenPH
Update matrix/max_area_of_island.py
8eb98e8 
Co-authored-by: Caeden Perelli-Harris <caedenperelliharris@gmail.com>
@kondekarshubham123 @CaedenPH
Update matrix/max_area_of_island.py
9b5338b 
Co-authored-by: Caeden Perelli-Harris <caedenperelliharris@gmail.com>
@kondekarshubham123
PR, comments resolved
3af0a52
@algorithms-keeper algorithms-keeper Bot added the tests are failing Do not merge until tests pass label Oct 17, 2022
cclauss
cclauss reviewed Oct 17, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
cclauss
cclauss reviewed Oct 17, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
@kondekarshubham123 @cclauss
Update matrix/max_area_of_island.py
094a4ce 
Co-authored-by: Christian Clauss <cclauss@me.com>
cclauss
cclauss reviewed Oct 17, 2022
View reviewed changes
Comment thread matrix/max_area_of_island.py Outdated
@kondekarshubham123 @cclauss
Update matrix/max_area_of_island.py
1f6464d 
Co-authored-by: Christian Clauss <cclauss@me.com>
@algorithms-keeper algorithms-keeper Bot removed the tests are failing Do not merge until tests pass label Oct 17, 2022
@kondekarshubham123
kondekarshubham123 requested review from cclauss and removed request for CaedenPH October 17, 2022 19:24
cclauss
cclauss approved these changes Oct 18, 2022
View reviewed changes

@cclauss cclauss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@cclauss

cclauss commented Oct 18, 2022

Copy link
Copy Markdown
Member

@cclauss
cclauss merged commit 2ca695b into TheAlgorithms:master Oct 18, 2022
@cclauss cclauss added hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest and removed awaiting reviews This PR is ready to be reviewed labels Oct 18, 2022
@kondekarshubham123
kondekarshubham123 deleted the matrix/max_area_of_island branch October 19, 2022 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CaedenPH CaedenPH CaedenPH left review comments
@cclauss cclauss cclauss approved these changes
+2 more reviewers
@CenTdemeern1 CenTdemeern1 CenTdemeern1 approved these changes
@kothekarakshata kothekarakshata kothekarakshata approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kondekarshubham123 @CenTdemeern1 @cclauss @CaedenPH @kothekarakshata