Skip to content

Conversation

@SammyUrfen
Copy link

@SammyUrfen SammyUrfen commented Oct 10, 2024

A good Hacktober PR

Summary

I added the Matrix operations directory to the repo with several python scripts, added scripts to Sorting folder and scripts to Star Pattern folder

Description

Added an entire directory named Matrix Operations with several scripts and a master matrix.py script. The purpose of this folder is to do matrix operations and transformations, there is also checker.py which checks for valid matrix, square matrix, if two matrices can be multiplied or not, if two matrices have the same dimensions or not, and an input.py script which takes a matrix input from user. Other scripts include functions functions for the operations themselves and the transformations (addition.py, subtraction.py, rotate.py, determinant.py, etc).
Added readme file for Scripts in Matrix Operations folder

Changes made to Sorting folder : Added three scripts (insertionsort-explanation.py, bubblesort-explanation.py, selectionsort-explanation.py) which prints all the steps explaining how the array was sorted using that sorting method
Added readme file for scripts in Sorting folder

Changes made to Star Pattern folder : Added several scripts for Pattern printing using * (stars)
Updated the readme file for better explaining the usage of the functions

Update the repo readme file to include my script
(the script link takes user to my forked repository. I request you to change it)

The changes are as follows:

  • Added Matrix Operation folder with several scripts
  • Added explanation scripts to Sorting folder
  • Added more Patterns to Star Pattern folder
  • updated master readme file to include my the added script

Checks

in the repository

  • [ x] Made no changes that degrades the functioning of the repository
  • Gave each commit a better title (unlike updated README.md)

in the PR

  • [ x] Followed the format of the pull_request_template
  • Made the Pull Request in a small level (for the creator's wellfare)
  • [ x] Tested the changes you made

Thank You,

Bibek Jyoti Charah

@hasan-py hasan-py requested a review from DhanushNehru October 10, 2024 16:15
Copy link
Owner

@DhanushNehru DhanushNehru left a comment

Choose a reason for hiding this comment

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

This seems to be a program and not a specific execution type script @SammyUrfen

@SammyUrfen
Copy link
Author

SammyUrfen commented Oct 10, 2024

Could you please elaborate on what you expected. I will make the necessary changes @DhanushNehru

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add matrix operations

2 participants