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
Add back xgboost.rabit for backwards compatibility #8408
Conversation
return collective.get_rank() | ||
|
||
|
||
def get_world_size() -> int: |
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.
get_world_size = collective.get_world_size
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.
And similarly for others. Not a request for change, just suggestions.
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.
Let's keep it as it is; the alternative seems less clear to me
* Add back xgboost.rabit for backwards compatibility * fix my errors * Fix lint * Use FutureWarning Co-authored-by: Hyunsu Philip Cho <chohyu01@cs.washington.edu>
Thanks everyone! |
Closes #8406 by making
xgboost.rabit
available in 1.7. Also add a warning to direct users to usexgboost.collective
instead.