-
Notifications
You must be signed in to change notification settings - Fork 192
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
print sccache stats in builds #2470
print sccache stats in builds #2470
Conversation
# Build ann-bench for each cuda and python version | ||
rapids-conda-retry mambabuild \ | ||
--no-test \ | ||
--channel "${CPP_CHANNEL}" \ | ||
--channel "${RAPIDS_CONDA_BLD_OUTPUT_DIR}" \ | ||
conda/recipes/raft-ann-bench | ||
|
||
sccache --show-adv-stats | ||
sccache --zero-stats | ||
|
||
# Build ann-bench-cpu only in CUDA 11 jobs since it only depends on python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(For a follow-up PR:) @cjnolet Do we still need to build raft-ann-bench
now that cuVS benchmarks are being packaged?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving once the if
block question is resolved.
/merge |
Description
Contributes to rapidsai/build-planning#111
Proposes some small packaging/CI changes, matching similar changes being made across RAPIDS.
sccache
stats to CI logspip
's verbosity in wheel building scriptsrapids-dependency-file-generator
(v1.16.0)cpp
/python
in calls torapids-upload-wheels-to-s3
Notes for Reviewers
This originally also ran wheel builds with
--no-build-isolation
, but I reverted that based on rapidsai/build-planning#108 (comment).