MathUtils: move BetheBlochAleph to a common header

[ktf]

No description provided.

[github-actions]

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

[sawenzel]

The codechecker on ARM seems to complain about more stuff than on X86.

[ktf]

Some of the checks depend on how the code is actually optimised, so that's architecture specific. PR updated with the fix.

[ktf]

@wiechula @davidrohr could you sign off this as TPC? Why was this not part of the common utils?

[ktf]

(or @shahor02, of course)

[davidrohr]

In general, I am fiine with this, but why do you keep the BetheBlochAleph.h header with just forwarding the function? I would say we should remove the header completely.

[ktf]

To avoid breaking O2Physics. We can clean it up once merged.

[ktf]

Is the error in dataflow:

Bulk-processing 2-prong Helix - Helix case
GPUassert: CUDA driver version is insufficient for CUDA runtime version /sw/SOURCES/O2/14918-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/GPUInterface.cu 51
unknown location(0): �[4;31;49mfatal error: in "DCAFitterNProngsBulk": std::runtime_error: GPU assert failed.�[0;39;49m
/sw/SOURCES/O2/14918-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/test/testDCAFitterNGPU.cxx(557): �[1;36;49mlast checkpoint: "DCAFitterNProngsBulk" test entry�[0;39;49m

a known issue?

[davidrohr]

Is the error in dataflow:

Bulk-processing 2-prong Helix - Helix case
GPUassert: CUDA driver version is insufficient for CUDA runtime version /sw/SOURCES/O2/14918-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/GPUInterface.cu 51
unknown location(0): �[4;31;49mfatal error: in "DCAFitterNProngsBulk": std::runtime_error: GPU assert failed.�[0;39;49m
/sw/SOURCES/O2/14918-slc9_x86-64/0/Common/DCAFitter/GPU/cuda/test/testDCAFitterNGPU.cxx(557): �[1;36;49mlast checkpoint: "DCAFitterNProngsBulk" test entry�[0;39;49m

a known issue?

Yes, I wrote on Mattermost to Sergio and you. It seems the dataflow container was changed, which enabled some GPU tests developed by Matteo. These tests never ran before, since the FullCI has ctests disabled to speed up the build. It seems these tests are generally broken, I tried locally on my system and they are failing as well.

#14935 disables them. Not merged yet since I discovered some more tests are failing and I want to get it fully clean.

We can merge the BetheBlockAleph PR if you like.