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

[R] Update docs for ... in predict #9868

Merged

Conversation

david-cortes
Copy link
Contributor

ref #9810

R method predict has a three-dots argument (...) which needs to be in the function signature of derived predict methods, such as xgboost's.

Currently, the docs for this parameter say:

Parameters passed to \code{predict.xgb.Booster}

But the parameters aren't actually used anywhere in the function.

Copy link
Member

@trivialfis trivialfis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix!

@trivialfis trivialfis merged commit ddab49a into dmlc:master Dec 12, 2023
24 of 28 checks passed
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

Successfully merging this pull request may close these issues.

2 participants