Skip to content

Pyhton Assignment 2 Submission#2

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

Pyhton Assignment 2 Submission#2
anisess10 wants to merge 1 commit intomainfrom
assignment-2

Conversation

@anisess10
Copy link
Copy Markdown
Owner

@anisess10 anisess10 commented Dec 7, 2024

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

Reading and displaying data from the first file to verify its structure and content.
Implementing the patient_summary function to compute summary statistics (mean, max...)

What did you learn from the changes you have made?

Understood how to efficiently compute summary statistics using NumPy's built-in functions

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

NA

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

Debugging logic in check_zeros to ensure accurate detection of zeros.

How were these changes tested?

For Part 1, displayed the contents of the file to visually verify the data structure.
For Part 2, validated the output shape of patient_summary (60 rows)
For Part 3, tested detect_problems with a dataset containing zeros to ensure accurate results.

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

NA

Checklist

  • I can confirm that my changes are working as intended

}
],
"source": [
"with open(r\"C:\\Users\\bouwa\\Desktop\\Anis Bwz\\Learning\\DSI\\Python\\python\\05_src\\data\\assignment_2_data\\inflammation_02.csv\", 'r') as f:\n",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@anisess10 Instead of only opening the inflammation_02.csv file, you may consider open all files (inflammation-01 to 12).
So, you may copy and run the all_paths =[".....","......",..........] code,
this is to define all_paths as those files (inflmmation-01 to 12), and then with open (all_paths[0], 'r').

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.

Great work!

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