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

ConfigureCUDA.cmake now sets CUVS_ prefixed variables #66

Conversation

robertmaynard
Copy link
Contributor

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

@robertmaynard
Copy link
Contributor Author

Ci failures fixed in rapidsai/raft#2254

@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 rapidsai/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 rapidsai/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
Copy link
Member

cjnolet commented May 17, 2024

/merge

@rapids-bot rapids-bot bot merged commit 11b3ecf into rapidsai:branch-24.06 May 17, 2024
54 checks passed
difyrrwrzd added a commit to difyrrwrzd/cuvs that referenced this pull request Aug 10, 2024
This makes sure that CUVS uses the compile flags that are required for RAPIDS C++ projects.

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

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

URL: rapidsai/cuvs#66
@robertmaynard robertmaynard deleted the bug/configure_cuda_set_correct_vars branch November 1, 2024 21:09
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
Development

Successfully merging this pull request may close these issues.

3 participants