Skip to content

UofT-DSI | Python - Assignment 2#2

Open
r0bay wants to merge 1 commit into
mainfrom
assignment-2
Open

UofT-DSI | Python - Assignment 2#2
r0bay wants to merge 1 commit into
mainfrom
assignment-2

Conversation

@r0bay
Copy link
Copy Markdown
Owner

@r0bay r0bay commented Apr 27, 2026

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

Reading in the CSV data and printing it out. Building the patient_summary function to get mean, max, and min values. Creating detect_problems to check if any patient has an average inflammation of 0

What did you learn from the changes you have made?

I learned how to actually work with CSV data using NumPy and how axis=1 applies operations across each patient. Also got more comfortable writing functions that process data instead of just printing stuff

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

No, I just followed the approach from the assignment and didn’t really consider another one

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

I ran into a NameError at first because I didn’t run the function cell before testing it. Fixed that by just running everything in order. Also had to wrap my head around how axis works in NumPy

How were these changes tested?

Ran the notebook cells and confirmed the outputs matched expected results, including checking that the output length was 60 and detect_problems returned False for the first file

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

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@monzchan monzchan left a comment

Choose a reason for hiding this comment

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

Excellent Work! That's Perfect

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