-
Notifications
You must be signed in to change notification settings - Fork 18
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
code and doc maintenance #133
Conversation
fixes #134 |
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 a lot, great job!
what about these three small changes
#' | ||
#' | ||
#' explain_titanic_rf <- explain(model_titanic_rf, | ||
#' data = titanic_imputed[,-8], | ||
#' y = titanic_imputed[,8], | ||
#' label = "Random Forest v7", |
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.
why this label is removed? it can be changed to Ranger v7
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.
I hope that "ranger forest"
is ok. (edited all examples)
vignettes/vignette_titanic.Rmd
Outdated
@@ -21,11 +21,11 @@ knitr::opts_chunk$set( | |||
# Data for Titanic survival | |||
|
|||
Let's see an example for `DALEX` package for classification models for the survival problem for Titanic dataset. | |||
Here we are using a dataset `titanic` avaliable in the `DALEX` package. Note that this data was copied from the `stablelearner` package. | |||
Here we are using a dataset `titanic_imputed` avaliable in the `DALEX` package. |
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.
let's keep here an information about the source (stablelearner) of the data.
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.
Information kept.
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.
ok, thanks
closes #130 closes #131 fixes #134
aut -> ctb
randomForest
->ranger
titanic
->titanic_imputed