Skip to content

Assignment 2#2

Open
ShadeRB wants to merge 6 commits intomainfrom
assignment-2
Open

Assignment 2#2
ShadeRB wants to merge 6 commits intomainfrom
assignment-2

Conversation

@ShadeRB
Copy link
Copy Markdown
Owner

@ShadeRB ShadeRB commented May 9, 2025

## What changes are you trying to make?
Adding the completed Jupyter Notebook assignment_2.ipynb for Assignment 2. The notebook includes analysis, function implementation, and file operations using relative paths.

What did you learn from the changes you have made?

I reinforced my understanding of file handling, summary statistics, and writing reusable functions in Python. I also learned how to use relative file paths in a project structure.

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

I considered using pandas directly for all summary operations, but chose to write custom functions as instructed.

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

Yes, I initially faced issues with incorrect file paths. I resolved it by understanding how relative paths work based on the location of the .ipynb file.

How were these changes tested?

I ran the notebook end-to-end in Jupyter Notebook and verified all output cells. I also validated that no file path errors occurred and all functions returned expected results.

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

N/A

Checklist

  • I can confirm that my changes are working as intended

@ShadeRB
Copy link
Copy Markdown
Owner Author

ShadeRB commented May 10, 2025

Hi! Just to clarify — the assignment_2.ipynb file is correctly located in the 02_activities/assignments/ folder, which is consistent with the existing structure in the main branch. This PR is only updating the file content, not moving it. Let me know if any adjustments are still needed. Thanks Ritu

@anjali-deshpande-hub
Copy link
Copy Markdown

Hi Ritu! I see two assignment_2.ipynb files. Please see https://github.com/ShadeRB/python/tree/assignment-2
The one inside assignments folder does not have the required changes. The assignment_2.ipynb under python directory has your changes.

@ShadeRB
Copy link
Copy Markdown
Owner Author

ShadeRB commented May 10, 2025

Assignment 2 has been updated with dynamic file handling, successful test runs, and no data file changes. All cells compile and execute as expected. Ready for review and merge.

Copy link
Copy Markdown

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

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