Skip to content

Assignment 2#2

Open
milassc wants to merge 2 commits into
mainfrom
assignment-2
Open

Assignment 2#2
milassc wants to merge 2 commits into
mainfrom
assignment-2

Conversation

@milassc
Copy link
Copy Markdown
Owner

@milassc milassc commented May 1, 2026

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

Added coding and reports.

What did you learn from the changes you have made?

How to work with files in Python and play around data in within the files.

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

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

There were some challenges in accessing list as array. converted the list to array and used it.

How were these changes tested?

Changes are tested with available data in the files provided in the repo

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

Checklist

  • [Yes] I can confirm that my changes are working as intended
  • [Yes] updated Assigment-2 python file

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!

"\n",
" # Implement the specific operation based on the 'operation' argument\n",
" summary_values = []\n",
" if operation == 'mean':\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.

You correctly computed the statistics, but the assignment expects a returned array rather than a printed table. Also, using NumPy’s vectorized operations makes the code faster and simpler. For example, in each 'if' clause, you only need:
if operation == 'mean':
summary_values = data.mean(axis=ax)

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