-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[R-package] silence more logs in unit tests #5227
Conversation
I see what looks like a bunch of networking-related issues across different R CI jobs.
And I see CRAN unavailable from my laptop as well. curl -I cran.r-project.org
# curl: (7) Failed to connect to cran.r-project.org port 80: Connection refused
curl -I http://cran.r-project.org
# curl: (7) Failed to connect to cran.r-project.org port 80: Connection refused
curl -I https://cran.r-project.org
# curl: (7) Failed to connect to cran.r-project.org port 443: Connection refused I think CRAN may be down. |
Yeah, it seems so. Still down... |
Looks like it is STILL down 😫 Dirk makes a good point in https://stat.ethz.ch/pipermail/r-devel/2022-May/081749.html ...other mirrors exist. I'm going to push a |
Just updated this to latest |
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
Contributes to #4862.
This PR proposes additional changes to silence logs in the R package's tests.
Notes for Reviewers
As of #5209, it's safe to change the verbosity in calls to
lightgbm()
without worrying about validation and early stopping.Using the following code, I found that this PR reduces the output of the R package's tests from 636 lines to 73 lines.