Skip to content

submission for assignment 2#2

Open
namathew7 wants to merge 2 commits intomainfrom
assignment-two
Open

submission for assignment 2#2
namathew7 wants to merge 2 commits intomainfrom
assignment-two

Conversation

@namathew7
Copy link
Owner

UofT-DSI | Python | Assignment Two

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

Added submission for assignment two.

What did you learn from the changes you have made?

Learned how to use some functions in the numpy library and how to do some basic data analysis of csv files.

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

Thought it would be cool to loop through the csvs in the folder and check each one instead of leading to a specific file path (though this is outside the scope of the assignment).

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

N/A

How were these changes tested?

Tested through changing the file paths and seeing the outputs change.

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

N/A

Checklist

  • [x ] I can confirm that my changes are working as intended

Copy link

@xindizhang xindizhang left a comment

Choose a reason for hiding this comment

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

Change Requested

Great work so far!

Part 1: Please complete the section "YOUR CODE HERE: Iterate through the variable using a for loop and print each row for inspection." — that part is currently missing.

Also, make sure you're processing the first file, as the question asks. Right now, your code is working with the second file instead.

Copy link

@xindizhang xindizhang left a comment

Choose a reason for hiding this comment

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

Thank you for the update. Great work — no further changes needed!

Minor note: for cleaner code, try putting your for loop to print each line inside the with block (the part where the file is opened and read). This way, everything related to reading the file stays together.

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