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

jenkins build for cuda 10.0 #4281

Merged
merged 2 commits into from
Mar 23, 2019
Merged

jenkins build for cuda 10.0 #4281

merged 2 commits into from
Mar 23, 2019

Conversation

rongou
Copy link
Contributor

@rongou rongou commented Mar 20, 2019

Also removed the hack to work around the cuda 10.1 compiler bug. We can try 10.1 again once update 1 is out and the bug is fixed.

@RAMitchell @trivialfis

@RAMitchell
Copy link
Member

The dockerfile searches for nccl binaries here (

wget https://developer.download.nvidia.com/compute/redist/nccl/v2.2/nccl_2.2.13-1%2Bcuda${CUDA_SHORT}_x86_64.txz && \
).

I'm not sure if 10.0 binaries are available at this link. We may have to move back to a nccl submodule to have a more sustainable way of building multi-GPU xgboost.

@rongou
Copy link
Contributor Author

rongou commented Mar 20, 2019

OK, I guess Jenkins doesn't actually use the Jenkinsfile in the PR?

Is there a reason to stick with CUDA 8 and Centos 6? If we drop CUDA 8 and move to Centos 7 there would be more options.

@RAMitchell
Copy link
Member

I think updates to the Jenkinsfile do not immediately take effect due to security reasons. Possibly it works for members, I'm not sure.

I set up the builds on Centos 6 and cuda 8.0 for maximum compatibility binary wheels (https://opensource.com/article/19/2/manylinux-python-wheels). Potentially we could upgrade cuda soon. What particular features do you need?

@hcho3
Copy link
Collaborator

hcho3 commented Mar 20, 2019

@RAMitchell I changed the security configuration for Jenkinsfile, so that changes are visible. Let me try again.

@hcho3 hcho3 closed this Mar 20, 2019
@hcho3 hcho3 reopened this Mar 20, 2019
@rongou
Copy link
Contributor Author

rongou commented Mar 22, 2019

Looks like Jenkins has passed with CUDA 10.0? Please take another look. @RAMitchell @hcho3

@hcho3 hcho3 merged commit 5aa42b5 into dmlc:master Mar 23, 2019
@hcho3
Copy link
Collaborator

hcho3 commented Mar 23, 2019

LGTM

@rongou rongou deleted the cuda-10 branch March 28, 2019 15:41
@lock lock bot locked as resolved and limited conversation to collaborators Jun 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants