Skip to content

Comments

Completed the assignment 2. Data display, summary calculator, and zer…#2

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

Completed the assignment 2. Data display, summary calculator, and zer…#2
SihanHe97 wants to merge 1 commit intomainfrom
assignment-2

Conversation

@SihanHe97
Copy link
Owner

…os checker. Sihan He

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

Added code to achieve three goals: 1. Display the content of the first checkin data. 2. Create a function to calculate the important basic statistics of the data (mean, maximum, minimum). 3. Create a problem detector function that detect irregular checkin entries (the ones containing zero means) based on the provided zero checker function.

What did you learn from the changes you have made?

Numpy is a very useful package that makes the code much more compact.

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

I thought of using for loop, which would make the code much longer and less manageable.

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

Not really.

How were these changes tested?

They were tested with appropriate inputs in the subsequent cells. I also identified that checkins 3 and 10 were irregular using the problem detector.

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

Checklist

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

Copy link

@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!

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