Skip to content

UofT-DSI | Python - Assignment 2#2

Merged
song-33 merged 5 commits intomainfrom
assignment-2
Nov 2, 2025
Merged

UofT-DSI | Python - Assignment 2#2
song-33 merged 5 commits intomainfrom
assignment-2

Conversation

@song-33
Copy link
Copy Markdown
Owner

@song-33 song-33 commented Oct 21, 2025

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

I added the code to print each row of inflammation_01.csv for inspection, defined the patient_summary function, and defined the detect_problems function using the check_zeros(x) helper function.

What did you learn from the changes you have made?

I learned how to read csv files into python, how to use for loops, and how to use functions in the numpy package.

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

I was thinking of using the count_nonzero(x) function in the numpy package to achieve a similar result as the helper function check_zeros(x).

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

I tried to use an if/else statement to return True/False depending on the value of check_zeros(x). However, I realized that I can just return check_zeros(x), as it is a boolean.

How were these changes tested?

I tested the changes with the examples provided in the assignment.

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

Checklist

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

Copy link
Copy Markdown

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Choose a reason for hiding this comment

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

Well done!

@song-33 song-33 merged commit 491e75e into main Nov 2, 2025
1 check passed
@song-33 song-33 deleted the assignment-2 branch November 2, 2025 03:28
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