Skip to content

Comments

Assignment 2#3

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

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

Conversation

@ottp613
Copy link
Owner

@ottp613 ottp613 commented Jan 12, 2026

UofT-DSI | LCR Assignment 2

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

data inspection, visualization, model initialize and model test

What did you learn from the changes you have made?

about data analysis, inspect, plots, understand the plots and correlate to a model

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

none

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

nothing specific

How were these changes tested?

locally

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

Checklist

  • I can confirm that my changes are working as intended

Copy link

@fyellow fyellow left a comment

Choose a reason for hiding this comment

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

Good Work! But you fit the model on the full dataset, not X_train. Could you please change that and resubmit?

@ottp613
Copy link
Owner Author

ottp613 commented Jan 16, 2026

@fyellow made the changes as per comments

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