Added the Strassen's Matrix Multiplication Algorithm by sourav-625 · Pull Request #13482 · TheAlgorithms/Python

sourav-625 · 2025-10-14T08:03:45Z

Describe your change:

Add an algorithm?
Fix a bug or typo in an existing algorithm?
Add or change doctests? -- Note: Please avoid changing both code and tests in a single pull request.
Documentation change?

Checklist:

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

…625/Python into strassen_matrix_multiply

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

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T08:03:52Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T08:03:52Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T08:03:52Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T08:05:02Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T08:05:02Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T08:05:03Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T08:10:43Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T08:10:43Z

matrix/strassen_matrix_multiply.py

+Matrix = list[list[int]]
+
+
+def add_matrices(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function add_matrices

algorithms-keeper · 2025-10-14T08:10:43Z

matrix/strassen_matrix_multiply.py

+    return [[matrix_a[i][j] + matrix_b[i][j] for j in range(size)] for i in range(size)]
+
+
+def subtract_matrices(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function subtract_matrices

algorithms-keeper · 2025-10-14T08:10:43Z

matrix/strassen_matrix_multiply.py

+    return [[matrix_a[i][j] - matrix_b[i][j] for j in range(size)] for i in range(size)]
+
+
+def multiply_matrices_naive(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function multiply_matrices_naive

algorithms-keeper · 2025-10-14T08:10:43Z

matrix/strassen_matrix_multiply.py

+    return result_matrix
+
+
+def get_next_power_of_two(n: int) -> int:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function get_next_power_of_two

Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T08:10:44Z

matrix/strassen_matrix_multiply.py

+    return top_left, top_right, bottom_left, bottom_right
+
+
+def join_matrix_quadrants(


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function join_matrix_quadrants

algorithms-keeper · 2025-10-14T08:10:44Z

matrix/strassen_matrix_multiply.py

+    return combined_matrix
+
+
+def strassen_matrix_multiplication(matrix_a: Matrix, matrix_b: Matrix, threshold: int = 64) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function strassen_matrix_multiplication

algorithms-keeper · 2025-10-14T08:10:44Z

matrix/strassen_matrix_multiply.py

+    return remove_matrix_padding(result_padded, original_size, original_size)
+
+
+def _strassen_recursive_multiply(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive_multiply

algorithms-keeper · 2025-10-14T08:10:44Z

matrix/strassen_matrix_multiply.py

+
+
+if __name__ == "__main__":
+    matrix_A = [[1, 2, 3], [4, 5, 6], [7, 8, 9]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: matrix_A

algorithms-keeper · 2025-10-14T08:10:44Z

matrix/strassen_matrix_multiply.py

+
+if __name__ == "__main__":
+    matrix_A = [[1, 2, 3], [4, 5, 6], [7, 8, 9]]
+    matrix_B = [[9, 8, 7], [6, 5, 4], [3, 2, 1]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: matrix_B

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

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T08:30:58Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T08:30:58Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T08:30:58Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T08:34:26Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T08:34:26Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T08:34:26Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T09:29:19Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T09:29:19Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T09:29:19Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-14T13:37:46Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-14T13:37:46Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper · 2025-10-14T13:37:46Z

matrix/strassen_matrix_multiply.py

+    return unpad_matrix(c_pad, n_orig, n_orig)
+
+
+def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:


As there is no test file in this pull request nor any test function or class in the file matrix/strassen_matrix_multiply.py, please provide doctest for the function _strassen_recursive

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-15T09:27:14Z

matrix/strassen_matrix_multiply.py

+https://en.wikipedia.org/wiki/Strassen_algorithm
+"""
+
+Matrix = list[list[int]]


Variable and function names should follow the snake_case naming convention. Please update the following name accordingly: Matrix

algorithms-keeper · 2025-10-15T09:27:14Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

algorithms-keeper

Click here to look at the relevant links ⬇️

🔗 Relevant Links

Repository:

Contributing guidelines

Project Euler solution guidelines

Python:

Formatted string literals (f-strings)

Type hints

doctest

unittest

pytest

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper commands and options

algorithms-keeper actions can be triggered by commenting on this PR:

@algorithms-keeper review to trigger the checks for only added pull request files

@algorithms-keeper review-all to trigger the checks for all the pull request files, including the modified files. As we cannot post review comments on lines not part of the diff, this command will post all the messages in one comment.

NOTE: Commands are in beta and so this feature is restricted only to a member or owner of the organization.

algorithms-keeper · 2025-10-15T09:38:50Z

matrix/strassen_matrix_multiply.py

+    return result
+
+
+def next_power_of_two(n: int) -> int:


Please provide descriptive name for the parameter: n

sourav-625 and others added 10 commits October 13, 2025 19:23

feat: Strassen's matrix multiplication algorithm added

3e55d21

feat: Strassen's matrix multiplication algorithm added

980a8b8

feat: Strassen's matrix multiplication algorithm added

d4a95a6

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

7889f30

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

feat: Strassen's matrix multiplication algorithm added

981f0c9

Merge branch 'strassen_matrix_multiply' of https://github.com/sourav-…

4c9e72a

…625/Python into strassen_matrix_multiply

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

6880ee5

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

feat: Strassen's matrix multiplication algorithm added

46ef515

feat: Strassen's matrix multiplication algorithm added

7d59f4c

feat: Strassen's matrix multiplication algorithm added

144024b

algorithms-keeper bot added require descriptive names This PR needs descriptive function and/or variable names require tests Tests [doctest/unittest/pytest] are required labels Oct 14, 2025

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

algorithms-keeper bot added the awaiting reviews This PR is ready to be reviewed label Oct 14, 2025

Merge branch 'master' into strassen_matrix_multiply

b1865cb

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

sourav-625 added 2 commits October 14, 2025 13:38

feat: Strassen's matrix multiplication algorithm added

8c6c2e4

feat: Strassen's matrix multiplication algorithm added

656b092

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

pre-commit-ci bot and others added 2 commits October 14, 2025 08:11

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

8a070b5

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

feat: Strassen's matrix multiplication algorithm added

5e951b6

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

feat: Strassen's matrix multiplication algorithm added

6724990

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

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 14, 2025

feat: Strassen's matrix multiplication algorithm added

4d7a808

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

algorithms-keeper bot removed the tests are failing Do not merge until tests pass label Oct 14, 2025

Merge branch 'master' into strassen_matrix_multiply

8aea982

algorithms-keeper bot reviewed Oct 14, 2025

View reviewed changes

feat: Strassen's matrix multiplication algorithm added

a337a94

algorithms-keeper bot removed the require tests Tests [doctest/unittest/pytest] are required label Oct 15, 2025

algorithms-keeper bot reviewed Oct 15, 2025

View reviewed changes

feat: Strassen's matrix multiplication algorithm added

c291794

algorithms-keeper bot reviewed Oct 15, 2025

View reviewed changes

feat: Strassen's matrix multiplication algorithm added

8db6205

algorithms-keeper bot added tests are failing Do not merge until tests pass and removed require descriptive names This PR needs descriptive function and/or variable names tests are failing Do not merge until tests pass labels Oct 15, 2025

sourav-625 added 3 commits October 15, 2025 20:23

Merge branch 'master' into strassen_matrix_multiply

ee1a8f3

Merge branch 'master' into strassen_matrix_multiply

3c4df5a

Merge branch 'master' into strassen_matrix_multiply

4fcb77e

		return unpad_matrix(c_pad, n_orig, n_orig)


		def _strassen_recursive(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:

		Matrix = list[list[int]]


		def add_matrices(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:

		return [[matrix_a[i][j] + matrix_b[i][j] for j in range(size)] for i in range(size)]


		def subtract_matrices(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:

		return [[matrix_a[i][j] - matrix_b[i][j] for j in range(size)] for i in range(size)]


		def multiply_matrices_naive(matrix_a: Matrix, matrix_b: Matrix) -> Matrix:

		return result_matrix


		def get_next_power_of_two(n: int) -> int:

		return top_left, top_right, bottom_left, bottom_right


		def join_matrix_quadrants(

		return combined_matrix


		def strassen_matrix_multiplication(matrix_a: Matrix, matrix_b: Matrix, threshold: int = 64) -> Matrix:

		return remove_matrix_padding(result_padded, original_size, original_size)


		def _strassen_recursive_multiply(matrix_a: Matrix, matrix_b: Matrix, threshold: int) -> Matrix:



		if __name__ == "__main__":
		matrix_A = [[1, 2, 3], [4, 5, 6], [7, 8, 9]]

Uh oh!

Comments

Conversation

sourav-625 commented Oct 14, 2025

Describe your change:

Checklist:

Uh oh!

algorithms-keeper bot left a comment

Choose a reason for hiding this comment

🔗 Relevant Links

Repository:

Python:

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper actions can be triggered by commenting on this PR:

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot left a comment

Choose a reason for hiding this comment

🔗 Relevant Links

Repository:

Python:

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper actions can be triggered by commenting on this PR:

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot left a comment

Choose a reason for hiding this comment

🔗 Relevant Links

Repository:

Python:

Automated review generated by algorithms-keeper. If there's any problem regarding this review, please open an issue about it.

algorithms-keeper actions can be triggered by commenting on this PR:

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

algorithms-keeper bot left a comment