-
Notifications
You must be signed in to change notification settings - Fork 197
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 GitHub Actions Workflows. #1076
Conversation
… to prevent the CUDA 11 "-dev" package from matching CUDA 12 runtime libraries due to its >= pinnings.
The sample-without-replacement test fails on A100 when `largeWeight` parameter is enabled, observed in PR #1076. The failure is caused as the sample with largest weight might not always be the first element getting selected in the sampled array. To fix this issue, this PR proposes to search the sample with large weight in the first three elements of sampled array. Authors: - Vinay Deshpande (https://github.com/vinaydes) Approvers: - Dante Gama Dessavre (https://github.com/dantegd) URL: #1109
`raft` doesn't have notebooks anyway
the `11.4.x` `cuda-profiler-api` package is only available for `aarch64`. therefore, this commit increases the upper limit of the version specifier for this package to avoid conda solver issues.
…ages This is to ensure that the other `raft` packages get built with the correct `libcu*` versioned packages. This commit also alphabetizes the lists.
this commit updates `dependencies.yaml` to: - include both `-dev` and non `-dev` packages of the `libcu*` packages - use a version range instead of a specific version pin for the `libcu*` packages
this moves `11.2` after `11.4` so that the `11.4` YAML anchors work with `11.2`
rerun tests |
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.
This looks great! I compared the old scripts and recipes w/ the new scripts and everything looks good to me.
When you guys are ready, I can help split the dependencies.yml
apart for the different recipes.
I think we'll be ready for that after this PR merges. Thanks again! |
@gpucibot merge |
This PR adds GitHub Actions workflows to
raft
.Wait until after the 22.12 release is complete (and potentially until other repos are ready to migrate to GHA) to merge.
Task list
Coverage required for this PR:
Result/18: SWoRTests/SWoRTestD; /opt/conda/conda-bld/work/cpp/test/random/sample_without_replacement.cu:269
Future work:
trap
function)rapids-dependency-file-generator
pre-commit hook