A few points about the introduction vignette that could be improved. I am looking at this version.
[x] the cfactuals$plot_surface(feature_names = c("betaplasma", "age"), grid_size = 200) plot is wider than the others.
[x] It is unfortunate that some of the output is wider than 96 chars, making it necessary to scroll horizontally; but we could avoid having our code be wider than 80 chars, e.g. the credit = german[, ... code at the top (but also others).
[x] text around the 'rchallenge' package is broken
[x] is it helpful to print class(cfactuals)?
[ ] Maybe explicitly mention that subset_to_valid() is an in-place operation, since R-users are not used to in-place operations.
[x] double space in cfactuals$plot_parallel()
[ ] mlr3-code: creating a Task is now easier, to get the plasma-task, you could just do as_task_regr(plasma[-100, ], target = "retplasma").
[x] mlr3-code is inconsistent about whether it uses mlr3:: or not
[x] consider writing the interval the interval [500, Inf] as [500, Inf)
A few points about the introduction vignette that could be improved. I am looking at this version.
cfactuals$plot_surface(feature_names = c("betaplasma", "age"), grid_size = 200)plot is wider than the others.credit = german[, ...code at the top (but also others).class(cfactuals)?subset_to_valid()is an in-place operation, since R-users are not used to in-place operations.cfactuals$plot_parallel()as_task_regr(plasma[-100, ], target = "retplasma").mlr3::or not