-
Notifications
You must be signed in to change notification settings - Fork 4
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
wheel builds: standardizations for 25.02 #136
Comments
jameslamb
changed the title
C++ wheel builds: standardizations for 25.02
wheel builds: standardizations for 25.02
Jan 22, 2025
rapids-bot bot
pushed a commit
to rapidsai/cugraph-gnn
that referenced
this issue
Jan 23, 2025
#111) Part of rapidsai/build-planning#136, which tracks some building/packaging simplifications and conventions we'd like to standardize across RAPIDS. This proposes the following: * using `cmake-format` to autoformat CMake code * using `cmake-lint` to enforce style preferences for CMake code * removing unnecessary use of `-DDETECT_CONDA_ENV` for wheel builds * explicitly passing package type to GitHub Actions / `gha-tools` things handling wheels ## Notes for Reviewers The `cmake-format` / `cmake-lint` approach was copied directly from RAFT: * https://github.com/rapidsai/raft/blob/596d4b7338e62a92652503cd76feaeaa187ad740/.pre-commit-config.yaml#L52 * https://github.com/rapidsai/raft/blob/596d4b7338e62a92652503cd76feaeaa187ad740/cpp/cmake/config.json * https://github.com/rapidsai/raft/blob/596d4b7338e62a92652503cd76feaeaa187ad740/cpp/scripts/run-cmake-format.sh Other RAPIDS projects ([like cuDF](https://github.com/rapidsai/cudf/blob/1f0f51f96b79edd820e81343ca521c684b1f4918/.pre-commit-config.yaml#L97)) do this the same way. All formatting-only changes to CMake in this PR were made automatically by `cmake-foramt`. Authors: - James Lamb (https://github.com/jameslamb) Approvers: - Gil Forsyth (https://github.com/gforsyth) - https://github.com/linhu-nv URL: #111
This was referenced Jan 23, 2025
rapids-bot bot
pushed a commit
to rapidsai/ucx-py
that referenced
this issue
Jan 23, 2025
Contributes to rapidsai/build-planning#136 This project's `update-version.sh` misses some RAPIDS dependencies in `pyproject.toml`. This fixes that. Also updates to the latest `rapids-dependency-file-generator`. ## Notes for Reviewers ### How I tested this ```shell git fetch upstream --tags ./ci/release/update-version.sh '0.43.00' git grep -E '25\.2' git grep -E '25\.02' git grep -E '0\.42' ``` Authors: - James Lamb (https://github.com/jameslamb) Approvers: - Jake Awe (https://github.com/AyodeAwe) URL: #1106
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
As more C++ wheels PRs (e.g. for #33) have gone through review, we've picked up a list of new preferred practices for RAPIDS wheels builds in CI. This tracks the work of going back through RAPIDS repos and applying all those suggestions.
-DDETECT_CONDA_ENV
in CMake flags for wheel builds (introduce libraft wheels raft#2531 (comment))rapids-cmake
style preferences viacmake-format
(introduce libraft wheels raft#2531 (comment))update-version.sh
(introduce libraft wheels raft#2531 (comment))wheel-build-*
jobs rely on otherwheel-build-*
jobs, notwheel-publish-*
(introduce libraft wheels raft#2531 (comment))wheel-build-libcudf
builds alibcudf
wheel and pushes it to S3...wheel-publish-libcudf
pushes that to the nightly pip index ...wheel-build-pylibcudf
just downloads from S3, so can start as soon aswheel-build-libcudf
is completerapids-dependency-file-generator
in pre-commit hooksBenefits of this work
Acceptance Criteria
Approach
Batch these changes into 1 PR per repo.
update-version.sh
can be tested like this:These can be done in any order.
Updates
The text was updated successfully, but these errors were encountered: