Skip to content

answers to assignment_1.ipynb#1

Open
Rollylake wants to merge 1 commit intomainfrom
assignment
Open

answers to assignment_1.ipynb#1
Rollylake wants to merge 1 commit intomainfrom
assignment

Conversation

@Rollylake
Copy link
Owner

@Rollylake Rollylake commented Dec 17, 2025

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

What did you learn from the changes you have made? I learnt the importance of standardizing the predictor variables

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of? I was not thinking of any other approach. The approach I used was fine.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it? Yes I had some challenges in the process of writing my codes. While running the codes, I had error messages, which I had to look into following the highlighted errors and using the help () function till I got it right.

How were these changes tested? I ran the codes again after figuring out the cause of the errors.

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

Checklist

  • I can confirm that my changes are working as intended

Copy link

@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)
When answering questions rather then commenting in after manually reading the output, should automate it. for eg.

Number of observations (rows)
num_observations = wine_df.shape[0]
print(f"Number of observations: {num_observations}")

@Rollylake
Copy link
Owner Author

Rollylake commented Dec 18, 2025 via email

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

Comments