microsoft / lightgbm-benchmark

Benchmark tools for LightGBM
MIT License
14 stars 7 forks source link

Add tests to MPIHandler #205

Closed jfomhover closed 2 years ago

jfomhover commented 2 years ago

Add missing tests for MPIHandler class in distributed.py Add tests to make sure script closes when distributed job fails with exception.

github-actions[bot] commented 2 years ago

Unit Test Results for Build

  1 files  ±0    1 suites  ±0   29s :stopwatch: ±0s 90 tests +2  90 :heavy_check_mark: +2  0 :zzz: ±0  0 :x: ±0 

Results for commit 0e2e297c. ± Comparison against base commit 201708d2.

This pull request removes 1 and adds 3 tests. Note that renamed tests count towards both. ``` tests.common.test_distributed ‑ test_multi_node_script_metrics ``` ``` tests.common.test_distributed ‑ test_mpi_handler tests.common.test_distributed ‑ test_multi_node_script tests.common.test_distributed ‑ test_multi_node_script_failure ```
github-actions[bot] commented 2 years ago

Code Coverage

Package Line Rate Branch Rate Complexity
common 85% 0% 0
scripts 100% 0% 0
scripts.data_processing.generate_data 93% 0% 0
scripts.data_processing.lightgbm_data2bin 95% 0% 0
scripts.data_processing.partition_data 92% 0% 0
scripts.inferencing.custom_win_cli 94% 0% 0
scripts.inferencing.lightgbm_c_api 75% 0% 0
scripts.inferencing.lightgbm_python 95% 0% 0
scripts.inferencing.treelite_python 94% 0% 0
scripts.model_transformation.treelite_compile 92% 0% 0
scripts.sample 93% 0% 0
scripts.training.lightgbm_python 80% 0% 0
Summary 86% (1223 / 1420) 0% (0 / 0) 0