Skip to content

Add rmm to pylibraft run dependencies, since it is used by Cython.#1656

Merged
rapids-bot[bot] merged 1 commit intorapidsai:branch-23.08from
bdice:add-rmm-to-pylibraft-run
Jul 19, 2023
Merged

Add rmm to pylibraft run dependencies, since it is used by Cython.#1656
rapids-bot[bot] merged 1 commit intorapidsai:branch-23.08from
bdice:add-rmm-to-pylibraft-run

Conversation

@bdice
Copy link
Contributor

@bdice bdice commented Jul 19, 2023

There is a build error in rapidsai/cugraph#3456:

$PREFIX/lib/python3.9/site-packages/pylibraft/common/handle.pxd:47:8: 'cuda_stream_view' is not a type identifier

pylibraft should have a run dependency on rmm so that Cython users of pylibraft (pylibcugraph in this case) can cimport and use pylibraft Cython at build time.

@bdice bdice requested a review from a team as a code owner July 19, 2023 21:07
@jakirkham jakirkham added bug Something isn't working non-breaking Non-breaking change labels Jul 19, 2023
@vyasr
Copy link
Contributor

vyasr commented Jul 19, 2023

/merge

@rapids-bot rapids-bot bot merged commit 0abedc6 into rapidsai:branch-23.08 Jul 19, 2023
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
…apidsai#1656)

There is a build error in rapidsai/cugraph#3456:
```
$PREFIX/lib/python3.9/site-packages/pylibraft/common/handle.pxd:47:8: 'cuda_stream_view' is not a type identifier
```

pylibraft should have a `run` dependency on `rmm` so that Cython users of pylibraft (pylibcugraph in this case) can `cimport` and use pylibraft Cython at build time.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)
  - AJ Schmidt (https://github.com/ajschmidt8)
  - https://github.com/jakirkham

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

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants