icecube / pisa

Monte Carlo-based data analysis
http://icecube.github.io/pisa/
Apache License 2.0
19 stars 47 forks source link

Multithreading fix #678

Closed atrettin closed 2 years ago

atrettin commented 2 years ago

Because the nuSQuIDS stage didn't check whether the TARGET was parallel, it would use multicore despite the target being CPU. We now check whether the target is parallel and in addition make sure that the global variable PISA_NUM_THREADS is always 1 if TARGET is cpu.

We also only give out the warning if the user tried to set PISA_NUM_THREADS in their environment explicitly larger than 1, but the TARGET is cpu.

philippeller commented 2 years ago

looks fine