Skip to content

UofT-DSI | <Python> - Assignment <2>#2

Open
MinsangKim-Data wants to merge 7 commits intomainfrom
assignment-2
Open

UofT-DSI | <Python> - Assignment <2>#2
MinsangKim-Data wants to merge 7 commits intomainfrom
assignment-2

Conversation

@MinsangKim-Data
Copy link
Copy Markdown
Owner

@MinsangKim-Data MinsangKim-Data commented Sep 8, 2024

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

I am adding code that overall evaluates patients medical records that evaluates the effectiveness of a fictional medication designed to reduce inflammation by arthritic flare-ups. Specifically I am trying to create a loop to ensure that the detect_problems function can evaluate each patient in the file list to make sure there are not any patients with zero mean inflammation scores.

What did you learn from the changes you have made?

I learned the usefulness of helper functions which makes other functions more readable and concise. In addition, I learned the power of using for loops for iteration.

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

I was thinking about instead of going through each file path, a function could read through all the file paths within a certain directory (all_paths). This would look like another for loop that would run over all_paths.

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

Some challenges were making sure that the file paths were connecting correctly and that the detect_problems function was running correctly for all of the file paths that I may test. I made sure by testing the function out with multiple different file paths and checking in my C drive that the original files did indeed have or not have zero mean.

How were these changes tested?

On top of checking manually and testing out different file paths, I also checked the loop structure and used f-string to check whether each file contained any patients with zero mean inflammation score.

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

Checklist

  • I can confirm that my changes are working as intended

@SidraBushra
Copy link
Copy Markdown

Assignment marked.

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