search

pecos / tps

Torch Plasma Simulator
BSD 3-Clause "New" or "Revised" License
8 stars 2 forks source link

Switch from cuda-memcheck to compute-sanitizer #289

Closed trevilo closed 2 months ago

trevilo commented 2 months ago

After OS upgrade on CI system, cuda-memcheck fails to detect the known errors in badcuda.cpp, which are based on examples from the nvidia docs:

https://docs.nvidia.com/cuda/archive/11.4.1/cuda-memcheck/index.html#example-use-of-memcheck

It isn't clear why this is the case, but since cuda-memcheck is deprecated in favor of compute-sanitizer in later versions anyway, I am switching to compute-sanitizer. It successfully detects errors in badcuda.

trevilo commented 2 months ago

PR also now includes a small update the the gitlab CI control, which was necessary because of changes to GitLab's runner authentication-token framework.