Skip to content
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

Review classification chapters #234

Merged
merged 36 commits into from
Sep 23, 2021
Merged

Review classification chapters #234

merged 36 commits into from
Sep 23, 2021

Conversation

leem44
Copy link
Contributor

@leem44 leem44 commented Aug 16, 2021

@ttimbers @trevorcampbell PR for classification chapters review!
addressing issues: #106, #107, #92 and #93 (I either addressed it, added a rebuttal or made a new issue)
Also addressed these: #184, #228, #205

Melissa Lee added 30 commits August 5, 2021 20:09
…more than 2 predictors sections with an example with 3 predictors, added explanation of set_engine, changed neighbours to american spelling
…ome paragraphs to clarify functions, removing unneeded explanations of functions which we will explain in earlier chapters, updating scaling/centering explanation, specifying data in balancing section
…kap from output, explaining confusion matrix
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification1.Rmd Outdated Show resolved Hide resolved
classification2.Rmd Outdated Show resolved Hide resolved
classification2.Rmd Outdated Show resolved Hide resolved
> **Note:** we set the seed when we call `train` not only because of the potential for ties, but also because we are doing
> cross-validation. Cross-validation uses a random process to select how to partition the training data.
> **Note:** we set the seed because we are doing
> cross-validation, which uses a random process to select how to partition the training data.
Copy link
Contributor

Choose a reason for hiding this comment

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

again, should be more careful about this note box

classification2.Rmd Outdated Show resolved Hide resolved
This was referenced Sep 22, 2021
@leem44 leem44 merged commit 28b7d32 into dev Sep 23, 2021
@trevorcampbell trevorcampbell deleted the review_classification1 branch December 1, 2021 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants