-
Notifications
You must be signed in to change notification settings - Fork 529
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
Split ensemble slides into bagging and boosting #471
Conversation
d6d6351
to
9a6b00e
Compare
Would be it worth doing related things in the same PR or is it better to leave it for a further PR, not sure ... Here is the kind of things I have in mind:
|
Co-authored-by: Olivier Grisel <[email protected]>
Co-authored-by: Olivier Grisel <[email protected]>
I started reviewing the private MR with the quiz answers: https://gitlab.inria.fr/learninglab/mooc-scikit-learn/mooc-scikit-learn-coordination/-/merge_requests/36 |
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.
LGTM!
Co-authored-by: Olivier Grisel <[email protected]> ac011b5
This PR adds the slides
bagging.md
andboosting.md
that result from splittingensemble.md
.Doing so required the refactoring of the quizzes in M6 and modifying the
toc.yml
file accordingly.It also adds spacing to the tables in the
custom.css
file.Note to ourselves: This PR has to be merged along with the respective gitlab MR.