The Wayback Machine - https://web.archive.org/web/20200831054406/https://github.com/TheAlgorithms/Python/pull/2310
Skip to content

/ Python

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

Create Transformations2D.py #2310

Merged
merged 4 commits into from Aug 15, 2020
Merged

Create Transformations2D.py #2310

merged 4 commits into from Aug 15, 2020
+62 −0

Conversation

@Yukti-09
Copy link
Contributor

Yukti-09 commented Aug 15, 2020

2D Transformations are regularly used in Linear Algebra.

I have added the codes for reflection, projection, scaling and rotation matrices.
@Yukti-09
Create Transformations2D.py
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
b88d555
@TravisBuddy
Copy link

TravisBuddy commented Aug 15, 2020

Hey @Yukti-09,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: a0058770-dee7-11ea-a7d8-3d177b5d5f1d
@Yukti-09
Update Transformations2D.py
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
dea2a39
@TravisBuddy
Copy link

TravisBuddy commented Aug 15, 2020

Hey @Yukti-09,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: 4ac99170-def1-11ea-a7d8-3d177b5d5f1d
@spamegg1
Copy link
Contributor

spamegg1 commented Aug 15, 2020

2 E225 missing whitespace around operator
11 E226 missing whitespace around arithmetic operator
4 E261 at least two spaces before inline comment
4 E262 inline comment should start with '# '
2 E265 block comment should start with '# '
2 E302 expected 2 blank lines, found 1
19 W191 indentation contains tabs
1 W291 trailing whitespace

These can be easily fixed by running black on your file before submission. Check out CONTRIBUTING.md for info on it.
@cclauss
Drop numpy and add type hints and doctests
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
e6aa3dc
@cclauss
cclauss approved these changes Aug 15, 2020
View changes
@cclauss
Rename Transformations2D.py to transformations_2d.py
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
391455e
@cclauss cclauss merged commit 14199e0 into TheAlgorithms:master Aug 15, 2020
2 checks passed
2 checks passed
codespell
Details
Travis CI - Pull Request Build Passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cclauss cclauss

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@Yukti-09 @TravisBuddy @spamegg1 @cclauss
You can’t perform that action at this time.