You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This project occasionally runs valgrind to check for memory management issues.
Those checks are currently run only on the code paths touched by the R package's tests. It generates a few false positives, and those are currently prevented from failing the relevant CI job via text-parsing in a shell script.
A valgrind suppression file should be used for that purpose instead.
Motivation
allows for finer-grained control over which errors from valgrind the project regards as noteworthy
makes the configuration portable outside of using R to run valgrind
Summary
This project occasionally runs
valgrind
to check for memory management issues.Those checks are currently run only on the code paths touched by the R package's tests. It generates a few false positives, and those are currently prevented from failing the relevant CI job via text-parsing in a shell script.
A
valgrind
suppression file should be used for that purpose instead.Motivation
valgrind
the project regards as noteworthyvalgrind
Description
To fix this, remove this stuff:
LightGBM/.ci/test_r_package_valgrind.sh
Lines 48 to 69 in 522f0f0
And instead check in a file,
.ci/valgrind.supp
, into the repo which suppresses those specific errors.To test, run the
valgrind
checks in a container, following https://github.com/microsoft/LightGBM/tree/master/R-package#valgrind.References
The text was updated successfully, but these errors were encountered: