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

Add wheel builds #1013

Merged
merged 125 commits into from
Nov 23, 2022
Merged

Add wheel builds #1013

merged 125 commits into from
Nov 23, 2022

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Nov 14, 2022

This PR enables building wheels for pylibraft and raft-dask.

@vyasr vyasr added 3 - Ready for Review and removed 2 - In Progress Currenty a work in progress labels Nov 18, 2022
@vyasr vyasr marked this pull request as ready for review November 18, 2022 23:48
@vyasr vyasr requested review from a team as code owners November 18, 2022 23:48
Copy link
Member

@cjnolet cjnolet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines 3 to 6
on:
push:
branches:
- 'pull-request/[0-9]+'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have the GPU resources currently to run the build / test workflows on every pull request. We'll need to either run a reduced build / test matrix for PRs or configure this workflow to run on a different event type.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been waiting for each PR to get approved before removing this so that everything up to the final set of changes is tested. Since it looks like we're ready I'll go ahead and update.

RAFT v22.12 Release automation moved this from PR-WIP to PR-Needs review Nov 21, 2022
@cjnolet
Copy link
Member

cjnolet commented Nov 21, 2022

rerun tests

@vyasr
Copy link
Contributor Author

vyasr commented Nov 21, 2022

rerun tests

1 similar comment
@vyasr
Copy link
Contributor Author

vyasr commented Nov 22, 2022

rerun tests

RAFT v22.12 Release automation moved this from PR-Needs review to PR-Reviewer approved Nov 22, 2022
@vyasr
Copy link
Contributor Author

vyasr commented Nov 23, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 4d0cdc3 into rapidsai:branch-22.12 Nov 23, 2022
RAFT v22.12 Release automation moved this from PR-Reviewer approved to Done Nov 23, 2022
@vyasr vyasr deleted the feat/cibuildwheel branch November 23, 2022 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants