Skip to content

Added installing-numpy.md file and updated the index.md #187

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 13 commits into from
Closed

Added installing-numpy.md file and updated the index.md #187

wants to merge 13 commits into from

Conversation

Yodha-Sudarsi
Copy link
Contributor

PR Description

Added installing-numpy.md file and updated the index.md

Related Issues

#78

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR

Undertaking

I declare that:

  1. The content I am submitting is original and has not been plagiarized.
  2. No portion of the work has been copied from any other source without proper attribution.
  3. The work has been checked for plagiarism, and I assure its authenticity.

I understand that any violation of this undertaking may have legal consequences that I will bear and could result in the withdrawal of any recognition associated with the work.

  • I Agree

@Yodha-Sudarsi
Copy link
Contributor Author

I'm really sorry @animator , Could you please keep the request open? This won't happen again. I realized I needed to change the PR description because we were having a conversation there. I changed the file accordingly after refering to #119

@Yodha-Sudarsi
Copy link
Contributor Author

Hi @animator, completed #78. I also completed the latest task #160. But the issue is I can only send pull request if you merge or close #78 after reviewing. Thank you and have a nice day man...

@animator
Copy link
Owner

File names should be - separated not _ separated.
You can create separate PRs. You just need to have a separate branch for each issue fix.
Also, for array operation, it should be - explain what the operator does followed by code example for the operator. Go though the learn python book to understand it better. Closing this PR. Raise two separate PRs fixing all the issues.

@animator animator closed this May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

installing numpy
2 participants
@Yodha-Sudarsi @animator