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

[ENH] Get rid of deprecated CUDA library calls #869

Closed
afender opened this issue May 13, 2020 · 5 comments
Closed

[ENH] Get rid of deprecated CUDA library calls #869

afender opened this issue May 13, 2020 · 5 comments
Assignees
Milestone

Comments

@afender
Copy link
Member

afender commented May 13, 2020

CuSparse is removing a lot of functions in the next major release https://docs.nvidia.com/cuda/cusparse/index.html.
cuGraph should be updated in order to build with the next major CUDA Toolkit version.

Many calls are related to spectral clustering which should be refactored and moved into RAFT #868

@afender afender added the ? - Needs Triage Need team to review and classify label May 13, 2020
@afender afender added this to the 0.16 milestone May 13, 2020
@BradReesWork BradReesWork changed the title [BUG] Get rid of deprecated CUDA library calls [ENH] Get rid of deprecated CUDA library calls May 26, 2020
@BradReesWork BradReesWork added Refactor and removed ? - Needs Triage Need team to review and classify labels May 26, 2020
@afender afender modified the milestones: 0.16, 0.15 Jun 8, 2020
@aschaffer
Copy link
Collaborator

I can switch the SpMv mechanism to use raft::sparse_matrix_t::mv(), which should also cover CUDA 11, but I need a raft::handle_t for that. Can I create one on-the-fly in the client code (i.e., cugraph code that calls SpMv, formerly CusparseCsrMV) ? Or, else, where do I get the raft handle from?

@aschaffer
Copy link
Collaborator

Also, I assume CusparseCsrMV is to be removed completely, correct?

@aschaffer
Copy link
Collaborator

It looks like the only remaining dependency is in src/utilities/spmv_1D.cu(h) via src/utilities/cusparse_helper.h(cu) on cusparseCsrmvEx(...). But that does not seem to be deprecated. In fact, I'm not sure if I should touch that code at all. No other dependencies remain: certainly no other cusparse or cublas calls. So, then should I replace/remove the above code, and if so with what? Some clarifications would help. Thanks in advance.

@aschaffer
Copy link
Collaborator

Completed. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants