Skip to content

UofT-DSI | Python - Assignment 2#2

Open
lantleUofT wants to merge 3 commits intomainfrom
assignment-2
Open

UofT-DSI | Python - Assignment 2#2
lantleUofT wants to merge 3 commits intomainfrom
assignment-2

Conversation

@lantleUofT
Copy link
Copy Markdown
Owner

UofT-DSI | Python - Assignment 2

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

  1. I loaded all lines from a patient into a nest list if lists (1 line in the csv --> 1 sublist), then I looped over all sub lists, printing them
  2. I completed a function by adding code to calculate the mean, minimum and maximum in a column wise manner for the csv loaded in 1 and return it as a numpy array
  3. I completed a function used the output of function 2 and input to function check_zeros returning a boolean true or false and then printing it

What did you learn from the changes you have made?

I applied our learning in class about numpy arrays, their methods and for loops

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?

NA

How were these changes tested?

The changes were testing using the test cases provided in the homework and adjusting the input file

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

NA

Checklist

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

@lantleUofT
Copy link
Copy Markdown
Owner Author

I loaded all lines from a patient into a nest list if lists (1 line in the csv --> 1 sublist), then I looped over all sub lists, printing them should be "I loaded all lines from a csv into a nested list of lists (1 line in the csv --> 1 sublist), then I looped over all sub lists, printing them"

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.

Approved

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.

3 participants