Skip to content

UofT-DSI | <Production> - Assignment 2#167

Closed
1SeanBrennan1 wants to merge 2 commits intoUofT-DSI:mainfrom
1SeanBrennan1:assignment-2
Closed

UofT-DSI | <Production> - Assignment 2#167
1SeanBrennan1 wants to merge 2 commits intoUofT-DSI:mainfrom
1SeanBrennan1:assignment-2

Conversation

@1SeanBrennan1
Copy link

@1SeanBrennan1 1SeanBrennan1 commented Jan 27, 2026

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

I tried changing the model, but the simpler model still one.

What did you learn from the changes you have made?

lots more iterations needed to figure out teh best model...but sometimes teh simplest is the best.

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

I would remove features to see if they are needed or not.

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

It was hard...but rewatched the classes, reread some of the book.

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

@github-actions
Copy link

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

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