Skip to content

Complete Assignment 2: Arthritis drug efficacy analysis#2

Open
YuouSheng wants to merge 1 commit intomainfrom
assignment-2
Open

Complete Assignment 2: Arthritis drug efficacy analysis#2
YuouSheng wants to merge 1 commit intomainfrom
assignment-2

Conversation

@YuouSheng
Copy link
Copy Markdown
Owner

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

Completed the Jupyter Notebook assignment_2.ipynb to analyze inflammation data and evaluate the drug’s efficacy. Implemented patient_summary(), check_zeros(), and detect_problems() functions and verified their outputs.

What did you learn from the changes you have made?

I learned how to use NumPy arrays to perform summary statistics and identify data irregularities. I also practiced working with file paths, functions, and modular code in Jupyter.

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

I considered manually looping through the CSV lines instead of using NumPy, but chose NumPy for cleaner and faster array operations.

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

I initially got None as output because I forgot to return a value in detect_problems(). After reviewing the logic, I fixed it by adding return result.

How were these changes tested?

I ran the notebook in Jupyter, executed all cells without errors, and verified the expected results:

  • The first inflammation file returns False
  • The functions behave as described in the assignment instructions

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

N/A

Checklist

  • 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! (Next time better git add only necessary files, not all of them :) )

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