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

Support CUDA 12.2 #4088

Merged
merged 33 commits into from
Feb 15, 2024
Merged

Support CUDA 12.2 #4088

merged 33 commits into from
Feb 15, 2024

Conversation

jameslamb
Copy link
Member

Description

  • switches to CUDA 12.2.2 for building conda packages and wheels
  • adds new tests running against CUDA 12.2.2

Notes for Reviewers

This is part of ongoing work to build and test packages against CUDA 12.2.2 across all of RAPIDS.

For more details see:

Planning a second round of PRs to revert these references back to a proper branch-24.{nn} release branch of shared-workflows once rapidsai/shared-workflows#166 is merged.

(created with rapids-reviser)

@github-actions github-actions bot added the conda label Jan 11, 2024
@jameslamb jameslamb marked this pull request as ready for review January 11, 2024 17:50
@jameslamb jameslamb requested a review from a team as a code owner January 11, 2024 17:50
@jameslamb jameslamb changed the title add CUDA 12.2 support for conda packages and wheels WIP: add CUDA 12.2 support for conda packages and wheels Jan 11, 2024
@jameslamb jameslamb marked this pull request as draft January 11, 2024 19:21
@jameslamb jameslamb changed the title WIP: add CUDA 12.2 support for conda packages and wheels WIP: use CUDA 12.2 for building and testing wheels Jan 11, 2024
@jameslamb jameslamb changed the title WIP: use CUDA 12.2 for building and testing wheels WIP: add CUDA 12.2 support for conda packages and wheels Jan 11, 2024
@jameslamb jameslamb changed the title WIP: add CUDA 12.2 support for conda packages and wheels WIP: (DO NOT MERGE) add CUDA 12.2 support for conda packages and wheels Jan 11, 2024
@jameslamb
Copy link
Member Author

one C API test is failing on CUDA 12.2 (ubunut 22.04, amd64)

Errors while running CTest

The following tests FAILED:
	 20 - CAPI_LEGACY_SPECTRAL_TEST (Failed)

(build link)

@jakirkham jakirkham added feature request New feature or request non-breaking Non-breaking change DO NOT MERGE Hold off on merging; see PR for details labels Jan 13, 2024
@jameslamb jameslamb changed the base branch from branch-24.02 to branch-24.04 January 22, 2024 15:36
@github-actions github-actions bot added the ci label Jan 23, 2024
@jakirkham
Copy link
Member

Depends on PR: rapidsai/raft#2179

@jakirkham
Copy link
Member

Retesting now that the aforementioned RAFT PR is merged and RAFT packages are available

@jakirkham
Copy link
Member

Still seeing old failure

Trying to restart again

@jameslamb jameslamb requested a review from a team as a code owner February 14, 2024 21:25
Comment on lines 272 to 274
// This test is skipped due to a bug in cuSparse 12.2 that affects RAFT.
// See https://github.com/rapidsai/raft/issues/2186
GTEST_SKIP();
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

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

Raised a cuGraph issue to track re-enabling this test

Suggested updates to the comments below to xref that cuGraph issue: #4173

cpp/tests/c_api/legacy_spectral_test.c Outdated Show resolved Hide resolved
cpp/tests/c_api/legacy_spectral_test.c Outdated Show resolved Hide resolved
cpp/tests/c_api/legacy_spectral_test.c Outdated Show resolved Hide resolved
Co-authored-by: jakirkham <jakirkham@gmail.com>
@jakirkham
Copy link
Member

Looks like there failures in the devcontainers

Comment on lines +740 to +747
#############################################################################
# Skipping due to CUDA 12.2 failure that traces back to RAFT #
# TODO: Uncomment this once the issue is fixed. #
# #
# xref: https://github.com/rapidsai/cugraph/issues/4173 #
# xref: https://github.com/rapidsai/raft/issues/2186 #
#############################################################################
# ConfigureCTest(CAPI_LEGACY_SPECTRAL_TEST c_api/legacy_spectral_test.c)
Copy link
Member

@jakirkham jakirkham Feb 14, 2024

Choose a reason for hiding this comment

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

Had trouble getting the GTEST_SKIPs Bradley added to work. So just commenting this for now. We can uncomment in a new PR to test fixes

xref: #4173
xref: rapidsai/raft#2186

@bdice
Copy link
Contributor

bdice commented Feb 15, 2024

/merge

@rapids-bot rapids-bot bot merged commit 8f31e43 into rapidsai:branch-24.04 Feb 15, 2024
109 checks passed
rapids-bot bot pushed a commit that referenced this pull request Feb 20, 2024
Follow-up to #4088

For all GitHub Actions configs, replaces uses of the `test-cuda-12.2` branch on `shared-workflows`
with `branch-24.04`, now that rapidsai/shared-workflows#166 has been merged.

### Notes for Reviewers

This is part of ongoing work to build and test packages against CUDA 12.2 across all of RAPIDS.

For more details see:

* rapidsai/build-planning#7

*(created with `rapids-reviser`)*

Authors:
  - James Lamb (https://github.com/jameslamb)
  - https://github.com/jakirkham

Approvers:
  - Ray Douglass (https://github.com/raydouglass)

URL: #4175
@jameslamb jameslamb deleted the test-cuda-12.2 branch June 17, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants