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

Feature Request: Add possibility so set default logging level for all mlr3-related loggers #1217

Open
sebffischer opened this issue Nov 27, 2024 · 3 comments

Comments

@sebffischer
Copy link
Member

sebffischer commented Nov 27, 2024

I find it somewhat annoying to having to silence different loggers and I can imagine it is also like this for users, for whom it's not even clear which loggers they have to change in order to get a less verbose output.

We could support an option mlr3.logging_level that is used by all packages from the mlr3 ecosystem.

@be-marc
Copy link
Member

be-marc commented Nov 27, 2024

I think we could solve this with inheritance https://cran.r-project.org/web/packages/lgr/vignettes/lgr.html#hierarchy. We could add a mlr3_ecosystem logger and the other logger inherit from it.

@be-marc
Copy link
Member

be-marc commented Nov 27, 2024

Might be a little bit tricky when bbotk is loaded without mlr3 but solvable.

@sebffischer
Copy link
Member Author

Maybe the base class goes in mlr3misc?

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

No branches or pull requests

2 participants