-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[docs] R docs cleanup #2375
[docs] R docs cleanup #2375
Conversation
Unfortunately, I have no R installed on my machine, so someone should update https://github.com/microsoft/LightGBM/tree/master/R-package/man folder before merging this PR. |
I can regenerate the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! @StrikerRUS I added bd75d6f to update the man/
folder. A few other small things in there that seem unrelated to your changes, seems they were not regenerated in past PRs.
@@ -646,7 +646,7 @@ Dataset <- R6::R6Class( | |||
#' @param colnames names of columns | |||
#' @param categorical_feature categorical features | |||
#' @param free_raw_data TRUE for need to free raw data after construct | |||
#' @param info a list of information of the lgb.Dataset object | |||
#' @param info a list of information of the \code{lgb.Dataset} object |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good change, thank you
@jameslamb Thank you very much!
Yeah, changes from #2176 were not regenerated. |
Result: https://lightgbm.readthedocs.io/en/r_examples/R/reference/index.html
Before:
After:
lgb.Dataset
andlgb.Booster
names.