Skip to content

LCR Assignment 1 Added#1

Open
milassc wants to merge 1 commit into
mainfrom
assignment-1
Open

LCR Assignment 1 Added#1
milassc wants to merge 1 commit into
mainfrom
assignment-1

Conversation

@milassc
Copy link
Copy Markdown
Owner

@milassc milassc commented May 23, 2026

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

Added Assignment-1 for LCR

What did you learn from the changes you have made?

Learned Classification using KNN modelling, standardization of data, Scaling etc.

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?

The response field 'class' was already removed in the dataset. I had to do some work around to add that field back and proceed.

How were these changes tested?

Executed and verified the results.

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

Checklist

  • [Yes] I can confirm that my changes are working as intended
  • [Yes] Added Assignemnt-1 file

Copy link
Copy Markdown

@PatelVishakh PatelVishakh left a comment

Choose a reason for hiding this comment

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

Assignment 1:Complete. Great work! Suggested Changes:


Q1)III) The type of variable is categorical. In a data science setting, this question is asking whether the variable is numerical or categorial (Integer, continuous, ordinal are for distinguishing further) to assess whether classification or regression methods should be used. The complete statement should Class is a Categorical Variable, represented here in integers (0,1,2) stored in our Dataframe as int64.

Q2)I)The more precise explanation is that Knn relies heavily on comparing distances between data points to fit a model to the data. Hence Variables with large scales sway the DISTANCE computation more, influencing the estimation quite incorrectly.
Vishakh Patel [LS]

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