[PWGHF] Adding D0/Lc analysis task - #17808
Conversation
Please consider the following formatting changes to AliceO2Group#17807
|
O2 linter results: ❌ 0 errors, |
|
Dear codeowners, Is it possible to overcome the "magic number" errors in the O2 linter / O2 linter (pull_request_target) check? In my opinion it would unnecessarily complicate the existing code. You can see an example of such a solution in the PWGLF/Tasks/GlobalEventProperties/flattenicityPikp.cxx file (lines 1814-1908) |
The way to "overcome" it is to write it in a better way. From what I can see, you map some number to other numbers. So how about using a map or an array, for example? |
Please consider the following formatting changes to AliceO2Group#17808
Please consider the following formatting changes to AliceO2Group#17808
Dear Vit, Thank you for the nice and easy solution to the issue. Everything should be fine now. |
Thanks @laszlogyulai for the implementation. It looks indeed much better now. |
|
Since you are adding a new file, please fix also the warnings. |
Please consider the following formatting changes to AliceO2Group#17808
|
Dear code owners, Can you tell me if anything is needed from my side at this point? |
You have not fixed the warnings. |
Co-authored-by: Vít Kučera <26327373+vkucera@users.noreply.github.com>
Co-authored-by: Vít Kučera <26327373+vkucera@users.noreply.github.com>
Adding a new analysis task for calculating the D0/Lambda_c yield ratio as a function of flattenicity