Use rlang::check_installed() on checking the dependencies

[yutannihilation]

Currently parsnip:::check_installs() just aborts when the dependency package is not installed yet. I think it would be helpful to use rlang::check_install() to let users install the packages interactively.

> rlang::check_installed(c("foo", "bar"))
ℹ The `foo` and `bar` packages are required.
x Would you like to install them?

1: Yes
2: No

Selection:

[topepo]

I think that it is better to leave as-is. If someone is using parallel processing during and interactive session, we don't want the worker processes trying to prompt the user for input.

If you want, you could create a check_package_dependency() function that will do the interactive bits outside of the tuning or resampling processes.

[yutannihilation]

Ah, I didn't consider parallelism, sorry...

[topepo]

No worries. It is was a good idea.

[juliasilge]

We could consider adding an additional "precheck" check before fitting happens (possibly in workers) to see if packages are installed in the main R process. This "precheck" (using rlang::check_installed() would probably be helpful and involve less friction for the majority of R users.

[simonpcouch]

In light of the above discussion, and for consistency with tune:::check_installs(), we'll continue to fail here. As long as we continue to fail early, I think this is indeed the right way to go.

[github-actions]

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.