-
Notifications
You must be signed in to change notification settings - Fork 521
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX Several small fixes #780
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, here are some further improvements:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some more feedback. LGTM otherwise.
@ArturoAmorQ I finally answered your feedback in #780 (comment). |
Co-authored-by: Olivier Grisel <[email protected]>
Thanks for the review @ogrisel. Merging! |
At the end of the First look at the dataset notebook, under the heading "we made important observations" we mention that linear models can only capture linear interactions but there is no prior reference to linear models.
The word "complex" at the end of the Effect of the sample size in cross-validation notebook may be misleading. Here we don’t mean complexity as in the depth of a decision tree or the degree of a polynomial (which indeed would lead to overfitting), but rather that a more "expressive" model should be used.
In the Solution for Exercise M3.02 the wording refers to the "accuracy of the model" but the model is a
KNeighborsRegressor
.The
value_counts
function in the Classification metrics notebook is raising a warning.This PR proposes some fixes to address those issues.