Skip to content

ConfigureCUDA.cmake now sets CUVS_ prefixed variables#66

Merged
rapids-bot[bot] merged 5 commits into
NVIDIA:branch-24.06from
robertmaynard:bug/configure_cuda_set_correct_vars
May 17, 2024
Merged

ConfigureCUDA.cmake now sets CUVS_ prefixed variables#66
rapids-bot[bot] merged 5 commits into
NVIDIA:branch-24.06from
robertmaynard:bug/configure_cuda_set_correct_vars

Conversation

@robertmaynard

Copy link
Copy Markdown
Contributor

This makes sure that CUVS uses the compile flags that are required for RAPIDS C++ projects.

@robertmaynard

Copy link
Copy Markdown
Contributor Author

Ci failures fixed in NVIDIA/raft#2254

Comment thread cpp/cmake/modules/ConfigureCUDA.cmake Outdated
@cjnolet cjnolet added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Apr 9, 2024
rapids-bot Bot pushed a commit to NVIDIA/raft that referenced this pull request Apr 10, 2024
The init order of class members needs to follow the order listed in the class. This corrects the error for the cagra index type, as it was found to be wrong via NVIDIA/cuvs#66

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: #2254
@robertmaynard robertmaynard force-pushed the bug/configure_cuda_set_correct_vars branch from 13167bc to 13ef7ec Compare May 14, 2024 15:06
@robertmaynard robertmaynard requested a review from a team as a code owner May 14, 2024 15:06
@cjnolet

cjnolet commented May 17, 2024

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot rapids-bot Bot merged commit 11b3ecf into NVIDIA:branch-24.06 May 17, 2024
@robertmaynard robertmaynard deleted the bug/configure_cuda_set_correct_vars branch November 1, 2024 21:09
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
The init order of class members needs to follow the order listed in the class. This corrects the error for the cagra index type, as it was found to be wrong via NVIDIA/cuvs#66

Authors:
  - Robert Maynard (https://github.com/robertmaynard)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: NVIDIA#2254
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake cpp improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants