Skip to content

I have updated the assignement -2#2

Open
vikrama33 wants to merge 2 commits intomainfrom
assignment-2
Open

I have updated the assignement -2#2
vikrama33 wants to merge 2 commits intomainfrom
assignment-2

Conversation

@vikrama33
Copy link
Copy Markdown
Owner

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

What did you learn from the changes you have made?

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?

How were these changes tested?

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

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@xindizhang xindizhang left a comment

Choose a reason for hiding this comment

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

Change Requested

Great exploration with os and file paths!

It looks like you’ve run into a path issue. Your Python folder is located in C:/Users/chait/ or C:/Users/chait/Desktop/, not C:/Users/chait/projects/dsi/. Please replace all instances of C:/Users/chait/projects/dsi/ with C:/Users/chait/ or C:/Users/chait/Desktop/in your code and give it another try.

You’re making great progress — keep it up!

Quick reminder for future submissions — including a brief PR description is good practice, as it helps your teammates quickly understand what’s going on.

Copy link
Copy Markdown

@xindizhang xindizhang 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! No changes needed

Nice to see you exploring different approaches! Just one minor suggestion: please clean up your code or add comments to indicate which part is your final answer, as it’s a little hard to spot right now.

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