Skip to content

assignment-2#2

Open
stella-hoang wants to merge 1 commit intomainfrom
assignment-2
Open

assignment-2#2
stella-hoang wants to merge 1 commit intomainfrom
assignment-2

Conversation

@stella-hoang
Copy link
Copy Markdown
Owner

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

I have added my code to the questions in assignment2.ipynb file.

What did you learn from the changes you have made?

I continued learning function definition, file reading, NumPy functions and problem detection.

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

I did various tests to check the expected behaviour and outputs which at times meant opening and verifying the file(s)' data manually through inspection, calculations, etc.

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

I encountered a syntax error and I had to retrace my steps to correct.
I also accidentally created this pull request to the UofT-DSI: main repo but have since closed it, please forgive me!

How were these changes tested?

These changes were tested by continually passing arguments with different inputs (e.g. swapped out min/max/mean and different file paths to double-check boolean value output).

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

Checklist

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

Copy link
Copy Markdown

@Aditya-k-23 Aditya-k-23 left a comment

Choose a reason for hiding this comment

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

Excellent submission and clean code! Your assignment is complete.

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