Skip to content

assignment_2 finished#2

Open
lyuxiaotian wants to merge 2 commits intomainfrom
assignment_2
Open

assignment_2 finished#2
lyuxiaotian wants to merge 2 commits intomainfrom
assignment_2

Conversation

@lyuxiaotian
Copy link
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?

avoiding the float when fitting the model

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

@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 2: Incomplete, Needs some changes.

Required changes:

Q1) I) Instead of commenting manually or just printing output, should make it part of a statement. for eg.
print(f"Number of observations (rows): {num_rows}, Number of variables (columns): {num_columns}")

Q1) II) More relevant response is Mpg is a continuous response variable stored in our dataset as float64 type.


Q2) I) Also Should explain what Positive, Negative and No association imply about the relationship with the response i.e. when model year increases on average the MPG goes down.

Q2) II) Should Also explain the plotted line represents the line of BEST FIT with respect to the mean of sum of squared errors. 



Q2) III) These differences are called residuals, and they show how far each data point is from the predicted value on the line.

@lyuxiaotian
Copy link
Owner Author

Hi, I've updated the assignment based on your comments.
Please let me know if any further changes are needed. Thank you!

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 2) Complete

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