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] Set up a nightly pipeline to test with dev versions of RAPIDS #10206

Closed
hcho3 opened this issue Apr 18, 2024 · 2 comments · Fixed by #10351
Closed

[CI] Set up a nightly pipeline to test with dev versions of RAPIDS #10206

hcho3 opened this issue Apr 18, 2024 · 2 comments · Fixed by #10351
Assignees

Comments

@hcho3
Copy link
Collaborator

hcho3 commented Apr 18, 2024

Currently, XGBoost is tested with latest stable version of RAPIDS. As a result, the latest XGBoost may not be fully compatible with an upcoming version of RAPIDS. By the time RAPIDS releases a new version, it may end up packaging a version of XGBoost that's not fully compatible with the new RAPIDS version. See #10175 for an example.

We should set up a CI pipeline that tests XGBoost with development version of RAPIDS on a nightly basis.

@hcho3 hcho3 self-assigned this Apr 18, 2024
@trivialfis
Copy link
Member

trivialfis commented May 30, 2024

Not entirely sure if we want to have nightly packages in the CI, we have very limited resources for handling sudden CI changes, considering issues can take time to resolve, particularly issues around distributed computing like spark, pyspark, and dask. Perhaps we can use integration tests that are run before release instead of unit tests in CI?

@hcho3
Copy link
Collaborator Author

hcho3 commented May 30, 2024

issues can take time to resolve

I think it's better if we can find issues earlier rather than wait until the release.

My suggestion is to continue using stable versions for testing pull requests and make a separate CI pipeline to test nightly version of Rapids.

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 a pull request may close this issue.

2 participants