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

[CI] Use RAPIDS nightly 21.10 #7251

Closed
wants to merge 1 commit into from
Closed

Conversation

hcho3
Copy link
Collaborator

@hcho3 hcho3 commented Sep 23, 2021

Ensure that XGBoost is compatible with the upcoming release of RAPIDS 21.10.

@trivialfis
Copy link
Member

There's a problem with the latest rmm. RMM uses a custom version of thrust and brings into include path of XGBoost. Currently I test the CUDA version, but not the thrust version.

@hcho3
Copy link
Collaborator Author

hcho3 commented Sep 23, 2021

@trivialfis So XGBoost isn't yet compatible with RMM 21.10 🤔

@trivialfis
Copy link
Member

That's correct. To get it working, we just need to dispatch the definition of discard iterator based on the thrust version. But I haven't got the time to look into them.

@hcho3
Copy link
Collaborator Author

hcho3 commented Sep 24, 2021

Superseded by #7254

@hcho3 hcho3 closed this Sep 24, 2021
@hcho3 hcho3 deleted the upgrade_rapids branch September 24, 2021 19:05
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.

None yet

2 participants