[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? |
|
@zhangbiao-phy @vkucera I have updated my local installation of O2Physics, and now the code is compiling and running on the latest version as well. In the end, it was missing the recent addition of the PVMults table to the HF codes. Thank you for your guidance in figuring everything out. |
|
Error while checking build/O2Physics/code-check for fa93653 at 2026-09-24 07:29: Full log here. |
|
@laszlogyulai Please fix the errors reported by the code-check. You can reproduce (and fix) the errors by following the documentation instructions. See https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check |
|
@zhangbiao-phy Please do not merge PRs before the tests finish. |
Please consider the following formatting changes to AliceO2Group#17808
|
@zhangbiao-phy can you please test this PR? |
What do you mean? |
Co-authored-by: Vít Kučera <26327373+vkucera@users.noreply.github.com>
Please consider the following formatting changes to AliceO2Group#17808
Adding a new analysis task for calculating the D0/Lambda_c yield ratio as a function of flattenicity