Skip to content

UofT-DSI | Python - Assignment 2#2

Open
msugar wants to merge 2 commits intomainfrom
assignment-2
Open

UofT-DSI | Python - Assignment 2#2
msugar wants to merge 2 commits intomainfrom
assignment-2

Conversation

@msugar
Copy link
Copy Markdown
Owner

@msugar msugar commented Aug 27, 2024

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Adding code for Assignment 2.

What did you learn from the changes you have made?

I've learned that:

  • The readlines() function to read from a file keeps the new lines at the end of each line.
  • When using NumPy, one needs to pay attention at the orientation of the data inside the file and choose the value of the axis parameter properly.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

  • I think in most situations it would be preferable to read a CSV file using Pandas, but I understand this assignment is focused on NumPy alone.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

  • No challenges.

How were these changes tested?

  • I've added code cells (marked with a "TEST" comment) to verify assertions on data values or stats, and/or display the content of variables for visual inspection of the data.

A reference to a related issue in your repository (if applicable)

N/A

Checklist

  • I can confirm that my changes are working as intended

@msugar msugar changed the title Submit code for assignment # 2 UofT-DSI | Python - Assignment 2 Aug 27, 2024
Copy link
Copy Markdown

@pedram-aliniaye-asli pedram-aliniaye-asli left a comment

Choose a reason for hiding this comment

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

You did a good job here!
Just a quick note: Please add comments to your code to make it clearer for other people and explain what is happening in your code.
Please go ahead and do the modifications and then close this pull request. Thanks!

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.

2 participants