ogrisel / pygbm

Experimental Gradient Boosting Machines in Python with numba.
MIT License
183 stars 32 forks source link

Thread scalability is suboptimal #38

Open ogrisel opened 6 years ago

ogrisel commented 6 years ago

As reported in https://github.com/ogrisel/pygbm/issues/30#issuecomment-436184680 , the scalability of pygbm is not as good as LightGBM.

Here are some results on a machine with the following CPUs:

Intel(R) Xeon(R) CPU E5-2650 v4 @ 2.20GHz: 2 sockets each with 12 cores each which means 48 hyperthreads in total.

1 thread (sequential)

NUMBA_NUM_THREADS=1 OMP_NUM_THREADS=1 python benchmarks/bench_higgs_boson.py  --n-trees 100 --learning-rate 0.1 --n-leaf-nodes 255
Model Time AUC Speed up
LightGBM 1045s 0.8282 1x
pygbm 1129s 0.8192 1x

8 threads

NUMBA_NUM_THREADS=8 OMP_NUM_THREADS=8 python benchmarks/bench_higgs_boson.py  --n-trees 100 --learning-rate 0.1 --n-leaf-nodes 255
Model Time AUC Speed up
LightGBM 160s 0.8282 6.53x
pygbm 356s 0.8193 3.2x

48 (hyper)threads

python benchmarks/bench_higgs_boson.py  --n-trees 100 --learning-rate 0.1 --n-leaf-nodes 255
Model Time AUC Speed up
LightGBM 91s 0.8282 11.5x
pygbm 130s 0.8193 8.7x

All of those pygbm runs used numba 0.40 from anaconda using the tbb backend (which is the fastest for pygbm).

ogrisel commented 5 years ago

Recent changes in master do parallelize more things and scalability is not as bad as reported anymore. pygbm tend to stay close to 1.5 of the duration of lightgbm at worst.

stuartarchibald commented 5 years ago

@ogrisel would it be worth taking a look with the new parallel diagnostics output http://numba.pydata.org/numba-doc/latest/user/parallel.html#diagnostics to check what is/isn't parallelized?

ogrisel commented 5 years ago

Thanks for the feedback @stuartarchibald. This code does many calls to several jitted functions. Is there a way to get all the diagnostic reports for all the functions jitted by numba at the end of the benchmark script?

stuartarchibald commented 5 years ago

Would setting the NUMBA_PARALLEL_DIAGNOSTICS environment variable work for that purpose?

ogrisel commented 5 years ago

Thanks, this is exactly what I was looking for. Sorry for not reading the doc carefully enough.

ogrisel commented 5 years ago

Actually what we really need is to do 2 runs under a profiler, one with NUMBA_NUM_THREADS=1 and one with NUMBA_NUM_THREADS=8 (for instance), and then for each numba function in the critical path, compute the speed up ratio and spot the functions that least benefit from parallel=True in term of speed up and then look at the detailed parallel diagnostic for those.

It's also possible that we have a function in the critical path that is not parallelized at all for some reason.

stuartarchibald commented 5 years ago

I think this is related https://github.com/numba/numba/issues/3438, as setting the thread count to one is not the same as just switching parallelism off (parallel transformations and scheduling still take place). There are potentially cases where adding more than one thread causes the code to slow down (parallel kernels with negligible per-thread work, but all the overhead of scheduling), and further kernels which cost more to schedule and execute on a thread than to just use the executing thread to run them.

ogrisel commented 5 years ago

I did a quick bench on the current master on a machine with many cores (without profiling for now):

NUMBA_NUM_THREADS=1 OMP_NUM_THREADS=1 python benchmarks/bench_higgs_boson.py --n-trees 100 --n-leaf-nodes 255 --learning-rate=0.5
Model Time AUC Speed up
LightGBM 431s 0.7519 1x
pygbm 460s 0.7522 1x
NUMBA_NUM_THREADS=8 OMP_NUM_THREADS=8 python benchmarks/bench_higgs_boson.py --n-trees 100 --n-leaf-nodes 255 --learning-rate=0.5
Model Time AUC Speed up
LightGBM 83s 0.7519 5.2x
pygbm 146s 0.7536 2.9x

Intel(R) Xeon(R) CPU E5-2660 v2 @ 2.20GHz with 2x10 physical cores.

This is not the same machine as previously but the scalability is still sub-optimal, so doing the profiling effort is required to understand where are the scalability bottlenecks.

Laurae2 commented 5 years ago

@ogrisel You can apply for a free Intel VTune license for profiling your code if you do research.

It will be much better than the numba profiler.

stuartarchibald commented 5 years ago

@Laurae2 I'm not sure what the "numba profiler" is, please could you clarify?! Numba has a built in parallel diagnostics tool which tracks transforms made to it's own IR of the Python source as it converts serial code to parallel code, but that's a compile-time diagnostic tool not a performance profiler.

Further, Numba 0.41.0 JIT profiling works with Intel Vtune, set the NUMBA_ENABLE_PROFILING environment variable to non-zero and that will register the LLVM JIT Event listener for Intel VTune.

Laurae2 commented 5 years ago

@stuartarchibald You can use the numba profiler here: https://github.com/numba/data_profiler (it just adds the signatures in reality). Incurs overhead penalty.

Still better to use Intel VTune for real profiling though (way more details and easier to pinpoint the issues).

stuartarchibald commented 5 years ago

@Laurae2 Ah, so that's what you are referring to, thanks. Yes, indeed, they have different purposes...

Laurae2 commented 5 years ago

@ogrisel Note that LightGBM number of threads scale with the number of columns. Higgs dataset does not have enough columns for 48 threads (it will underestimate the scalability which gives you a lower scaling target).