cuvs icon indicating copy to clipboard operation
cuvs copied to clipboard

Test RAFT with CCCL mdspan

Open bdice opened this issue 3 months ago • 4 comments

Testing https://github.com/rapidsai/raft/pull/2836/.

bdice avatar Oct 14 '25 22:10 bdice

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

copy-pr-bot[bot] avatar Oct 14 '25 22:10 copy-pr-bot[bot]

/ok to test

bdice avatar Oct 14 '25 22:10 bdice

/ok to test

bdice avatar Oct 16 '25 06:10 bdice

The build errors here seem to stem from a bunch of places in the source code where the extents type is left as default (uint32_t) or defined incorrectly. The previous mdspan implementation was willing to implicitly convert something like raft::make_device_matrix_view<const int64_t, uint64_t> to raft::make_device_matrix_view<const int64_t, int64_t>, unsafely casting uint64 to int64 in the extents type. The CCCL mdspan does not allow that. I am iterating through build errors to ensure that we have extent types properly defined.

bdice avatar Oct 17 '25 18:10 bdice