Skip to content

Assignment 2#3

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

Assignment 2#3
ltonello wants to merge 3 commits intomainfrom
assignment-2

Conversation

@ltonello
Copy link
Copy Markdown
Owner

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

Added code that read file data, computes different operations (mean, max, etc.) and finds potential data problems

What did you learn from the changes you have made?

Learned how to read data files and perform operations and functions on arrays

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

Nope

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

Not too many challenges with this assignment

How were these changes tested?

Changes were tested by running defined functions on sample data

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

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

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

The code in assignment 2 looks good. But there just one thing to fix. This PR has changes to assignment_1.ipynb file as well. It should only contain changes to assignment_2.ipynb. Please follow this thread in slack (https://uoft-dsi-certificates.slack.com/archives/C07VC7M4PTL/p1733888327768759) to fix this isue.

@ltonello
Copy link
Copy Markdown
Owner Author

I think I fixed it - please let me know if I did it correctly!

Copy link
Copy Markdown

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

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

Looks good!

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