• James Lamb's avatar
    [ci] [R-package] Fix memory leaks found by valgrind (#3443) · 81d76113
    James Lamb authored
    
    
    * fix int64 write error
    
    * attempt
    
    * [WIP] [ci] [R-package] Add CI job that runs valgrind tests
    
    * update all-successful
    
    * install
    
    * executable
    
    * fix redirect stuff
    
    * Apply suggestions from code review
    Co-authored-by: default avatarGuolin Ke <guolin.ke@outlook.com>
    
    * more flags
    
    * add mc to msvc proj
    
    * fix memory leak in mc
    
    * Update monotone_constraints.hpp
    
    * Update r_package.yml
    
    * remove R_INT64_PTR
    
    * disable openmp
    
    * Update gbdt_model_text.cpp
    
    * Update gbdt_model_text.cpp
    
    * Apply suggestions from code review
    
    * try to free vector
    
    * free more memories.
    
    * Update src/boosting/gbdt_model_text.cpp
    
    * fix using
    
    * try the UNPROTECT(1);
    
    * fix a const pointer
    
    * fix Common
    
    * reduce UNPROTECT
    
    * remove UNPROTECT(1);
    
    * fix null handle
    
    * fix predictor
    
    * use NULL after free
    
    * fix a leaking in test
    
    * try more fixes
    
    * test the effect of tests
    
    * throw exception in Fatal
    
    * add test back
    
    * Apply suggestions from code review
    
    * commet some tests
    
    * Apply suggestions from code review
    
    * Apply suggestions from code review
    
    * trying to comment out tests
    
    * Update openmp_wrapper.h
    
    * Apply suggestions from code review
    
    * Update configure
    
    * Update configure.ac
    
    * trying to uncomment
    
    * more comments
    
    * more uncommenting
    
    * more uncommenting
    
    * fix comment
    
    * more uncommenting
    
    * uncomment fully-commented out stuff
    
    * try uncommenting more dataset tests
    
    * uncommenting more tests
    
    * ok getting closer
    
    * more uncommenting
    
    * free dataset
    
    * skipping a test, more uncommenting
    
    * more skipping
    
    * re-enable OpenMP
    
    * allow on OpenMP thing
    
    * move valgrind to comment-only job
    
    * Apply suggestions from code review
    Co-authored-by: default avatarNikita Titov <nekit94-08@mail.ru>
    
    * changes from code review
    
    * Apply suggestions from code review
    Co-authored-by: default avatarNikita Titov <nekit94-08@mail.ru>
    
    * linting
    
    * issue comments too
    
    * remove issue_comment
    Co-authored-by: default avatarGuolin Ke <guolin.ke@outlook.com>
    Co-authored-by: default avatarNikita Titov <nekit94-08@mail.ru>
    81d76113
r_valgrind.yml 955 Bytes