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.

Sign up for GitHub

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

Jump to bottom

Add wheel builds #2964

Merged
merged 162 commits into from
Nov 30, 2022
Merged

Add wheel builds #2964

merged 162 commits into from
Nov 30, 2022

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Nov 19, 2022 •
edited
Loading

This PR adds wheel building to cugraph.

Also resolves #3004.

sevagh and others added 30 commits September 26, 2022 08:35
@sevagh
Add initial cibuildwheel workflow
e51b1af
@sevagh
Install libraft for pycugraph/pylibcugraph
9268a6a
@sevagh
Install rmm
259df35
@sevagh
Install libcugraph-ops
a55e465
@sevagh
Install raft
2fbccfb
@sevagh
Add libcugraph_c to targets
60ef7ef
@sevagh
Add python cuda suffix
1cd1821
@sevagh
Adjust cugraph_c target
d08ac8c
@sevagh
DRY workflow with testing
5354078
@sevagh
Add unit test commands for cugraph/pylibcugraph
61473e9
@sevagh
Re-run without always
422e979
@sevagh
Switch from -cuda11 to -cu11 suffix
3d48832
@trxcllnt @sevagh
WIP
121bbcb
@trxcllnt @sevagh
WIP
011b491
@trxcllnt @sevagh
ensure cuGraph statically links all CTK libraries, not just cudart
ee0c828
@trxcllnt @sevagh
use correct version for finding cugraph
bb689e8
@trxcllnt @sevagh
only publicly link raft if not linking statically
614f738
@trxcllnt @sevagh
get pylibcugraph pypi wheels building
535acac
@trxcllnt @sevagh
get cugraph pypi wheels building
cb15133
@trxcllnt @sevagh
add dask-cuda to install_requires
04ad685
@trxcllnt @sevagh
set include_package_data=True for scikit-build, make raft-dask a depe…
e766d26 
…ndency of cugraph wheel
@trxcllnt @sevagh
update github action to only build pylibcugraph and cugraph wheels
5c63c7d
@trxcllnt @sevagh
put envvars on one line
635c804
@trxcllnt @sevagh
allow overriding cugraph-ops branch and repo url
16783f2
@trxcllnt @sevagh
add missing gtest link
e4d904d
@trxcllnt @sevagh
allow overriding the raft branch and repo url
859c5d4
@trxcllnt @sevagh
define extras_require
33b4a86
@trxcllnt @sevagh
cd into the correct directory
480878e
@trxcllnt @sevagh
can't reference secrets in this file when the workflow_call action in…
9bcbbde 
…herits them
@sevagh
Update to new workflow syntax
7f9df7e
@vyasr vyasr marked this pull request as ready for review November 23, 2022 18:25
@vyasr vyasr requested review from a team as code owners November 23, 2022 18:25
@BradReesWork
Copy link
Member

rerun tests

rlratzel
rlratzel reviewed Nov 29, 2022
View reviewed changes
Copy link
Contributor

@rlratzel rlratzel 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 very much for this, these changes look good and also clean up some tech debt. I did have a couple of questions though.

cpp/CMakeLists.txt Show resolved Hide resolved
cpp/cmake/thirdparty/get_libcugraphops.cmake Outdated Show resolved Hide resolved
@BradReesWork
Copy link
Member

rerun tests

@vyasr vyasr mentioned this pull request Nov 29, 2022
Closed
@vyasr
Copy link
Contributor Author

vyasr commented Nov 29, 2022

rerun tests

1 similar comment
@BradReesWork
Copy link
Member

rerun tests

@rlratzel rlratzel mentioned this pull request Nov 30, 2022
Closed
@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 70ae668 into rapidsai:branch-22.12 Nov 30, 2022
@vyasr vyasr deleted the feat/cibuildwheel branch November 30, 2022 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jolorunyomi jolorunyomi jolorunyomi approved these changes

@BradReesWork BradReesWork BradReesWork approved these changes

@rlratzel rlratzel rlratzel approved these changes

Assignees

@vyasr vyasr

Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Milestone
22.12
Development

Successfully merging this pull request may close these issues.

Add ALLOW_CLONE_CUGRAPH_OPS option to CMakeLists.txt
7 participants
@vyasr @codecov-commenter @BradReesWork @rlratzel @jolorunyomi @sevagh @trxcllnt