[ENHANCEMENT]: Add compute-sanitizer tests
Open
Nobody has claimed this yet.
P3: Backlog
topic: CI
type: feature request
- Dominant language
- Cuda
- Stars
- 667
- Forks
- 120
- Avg merge
- 7d 5h
- Merged PRs (30d)
- 4
Description
Is your feature request related to a problem? Please describe.
#256 showed that we might want to include tests to check for memory leaks or similar problems.
Describe the solution you'd like
We should implement a test that runs compute-sanitizer --tool memcheck similar to how cudf solves this: https://github.com/rapidsai/cudf/blob/d6582a1ef2dea745d81a24ccfaf74f03aa211666/ci/gpu/build.sh#L250
Describe alternatives you've considered
No response
Additional context
No response
Contributor guide
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
Review the cuDF build.sh example linked in the issue, then inspect this repository’s existing test and CI entry points to find where a sanitizer run belongs. Done means a test invokes compute-sanitizer --tool memcheck and reports failures.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- ci-cd, testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100