[TASK] Clean up ctk lib "wrapper" calls in cuml
Nobody has claimed this yet.
- Dominant language
- Cuda
- Stars
- 1k
- Forks
- 251
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 31
Description
We've been moving the wrapper functions for the ctk libraries into detail so that we can better manage the RAFT API functions which are being invoked by consuming projects.
Step 1 was to hide the wrapper functions in the detail namespace and Step 2 is to re-route all calls to functions in detail from downstream projects so they are calling into the public API.
Additionally, the cublas wrapper functions in linalg have any unnecessary overloads, which can be consolidated. These will also need to continue to provide the option of accepting device or host pointers for scalar parameters alpha and beta, in which the new device_scalar_view will be useful.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the ctk wrapper functions moved into the detail namespace and the downstream calls that still use them. Review the cublas wrapper functions in linalg alongside device_scalar_view; done means calls use the public API and unnecessary overloads are consolidated while preserving host and device scalar support.
Written by the indexing model from the issue text.
Assessment
- Domain
- api, backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100