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

[FR]: check_installed0 #1757

Closed
mikmart opened this issue Oct 22, 2024 · 2 comments
Closed

[FR]: check_installed0 #1757

mikmart opened this issue Oct 22, 2024 · 2 comments

Comments

@mikmart
Copy link
Contributor

mikmart commented Oct 22, 2024

The user-friendly errors and prompting to install are great features in check_installed(). However, as seen in #1608, the check to see if a package is in fact installed is almost 3 orders of magnitude slower than the simple requireNamespace().

Would there be scope for a check_installed0() that would retain the errors and prompting of check_installed(), but use a less featureful method of detecting whether a package is installed or not, allowing better performance? That would be useful for a package providing an optional feature in a fast but frequently called function that needs to check whether an optional dependency is available.

@lionel-
Copy link
Member

lionel- commented Oct 23, 2024

Agreed the current situation is not ideal. We should try and optimise check_installed() first, so let's track this in #1608.

@lionel- lionel- closed this as not planned Won't fix, can't repro, duplicate, stale Oct 23, 2024
@mikmart
Copy link
Contributor Author

mikmart commented Oct 23, 2024

That's fair. Thanks for considering!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants