[PWGHF] feat(taskLcToK0sP): add THnSparse with ML scores, align with taskLc patterns

[dongfangqingxin]

Add THnSparse support for Λc→K0sP analysis with BDT scores (Data, MC Rec, MC Gen). Add process functions for standard/ML with FT0C/FT0M centrality estimators. Refactor code structure to align with taskLc.cxx."

[github-actions]

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

[vkucera]

Fix the reported issues.

[vkucera]

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

[dongfangqingxin]

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

[vkucera]

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

You don't need to install and run it separately, if you're using the pre-commit hooks.

[dongfangqingxin]

@dongfangqingxin Thanks for the fixes. You have overwritten the formatting changes four times. Please learn how to update your local branch and/or how to format your local changes automatically to avoid it.

Hi @vkucera Thanks for the heads-up and sorry for the back-and-forth. I've installed clang-format locally now, will make sure to run it before pushing next time.

You don't need to install and run it separately, if you're using the pre-commit hooks.

Ah, I see — thanks for the tip! I hadn't set up the pre-commit hooks. I'll activate them now to avoid this in the future.

[zhangbiao-phy]

Thanks @dongfangqingxin! The changes are good to me!

[vkucera]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

[dongfangqingxin]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

Hi @vkucera, Thanks for your comments. I didn't get these warnings on my machine, but I believe they can be fixed by adding the following includes:

#include <cstdint>                               // for int8_t
#include "Common/DataModel/Centrality.h"         // for CentFT0Cs, CentFT0Ms
#include "Common/DataModel/EventSelection.h"     // for EvSels

I will push a commit and see if the CI passes. Please let me know if you have any further suggestions.

[vkucera]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

Hi @vkucera, Thanks for your comments. I didn't get these warnings on my machine, but I believe they can be fixed by adding the following includes:

#include <cstdint>                               // for int8_t
#include "Common/DataModel/Centrality.h"         // for CentFT0Cs, CentFT0Ms
#include "Common/DataModel/EventSelection.h"     // for EvSels

I will push a commit and see if the CI passes. Please let me know if you have any further suggestions.

How did you verify the list of includes?

[dongfangqingxin]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

Hi @vkucera, Thanks for your comments. I didn't get these warnings on my machine, but I believe they can be fixed by adding the following includes:

#include <cstdint>                               // for int8_t
#include "Common/DataModel/Centrality.h"         // for CentFT0Cs, CentFT0Ms
#include "Common/DataModel/EventSelection.h"     // for EvSels

I will push a commit and see if the CI passes. Please let me know if you have any further suggestions.

How did you verify the list of includes?
Hi @vkucera, Thank you for the reminder.
I cross-checked with existing tasks in the same directory, such as PWGHF/D2H/Tasks/taskLc.cxx and taskCharmPolarisation.cxx, and also located the necessary tables in Common/DataModel/Centrality.h and Common/DataModel/EventSelection.h.

[vkucera]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

Hi @vkucera, Thanks for your comments. I didn't get these warnings on my machine, but I believe they can be fixed by adding the following includes:

#include <cstdint>                               // for int8_t
#include "Common/DataModel/Centrality.h"         // for CentFT0Cs, CentFT0Ms
#include "Common/DataModel/EventSelection.h"     // for EvSels

I will push a commit and see if the CI passes. Please let me know if you have any further suggestions.

How did you verify the list of includes?
Hi @vkucera, Thank you for the reminder.
I cross-checked with existing tasks in the same directory, such as PWGHF/D2H/Tasks/taskLc.cxx and taskCharmPolarisation.cxx, and also located the necessary tables in Common/DataModel/Centrality.h and Common/DataModel/EventSelection.h.

That is not how you are supposed to do that. Follow the instructions. https://aliceo2group.github.io/analysis-framework/docs/tools/#cleaning-include-statements-and-using-statements

[dongfangqingxin]

You are supposed to verify your includes.

PWGHF/D2H/Tasks/taskLcToK0sP.cxx:71:54: warning: no header providing "o2::aod::EvSels" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:73:75: warning: no header providing "o2::aod::CentFT0Cs" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:75:75: warning: no header providing "o2::aod::CentFT0Ms" is directly included [misc-include-cleaner]
PWGHF/D2H/Tasks/taskLcToK0sP.cxx:476:11: warning: no header providing "int8_t" is directly included [misc-include-cleaner]

Hi @vkucera, Thanks for your comments. I didn't get these warnings on my machine, but I believe they can be fixed by adding the following includes:

#include <cstdint>                               // for int8_t
#include "Common/DataModel/Centrality.h"         // for CentFT0Cs, CentFT0Ms
#include "Common/DataModel/EventSelection.h"     // for EvSels

I will push a commit and see if the CI passes. Please let me know if you have any further suggestions.

How did you verify the list of includes?
Hi @vkucera, Thank you for the reminder.
I cross-checked with existing tasks in the same directory, such as PWGHF/D2H/Tasks/taskLc.cxx and taskCharmPolarisation.cxx, and also located the necessary tables in Common/DataModel/Centrality.h and Common/DataModel/EventSelection.h.

That is not how you are supposed to do that. Follow the instructions. https://aliceo2group.github.io/analysis-framework/docs/tools/#cleaning-include-statements-and-using-statements

Hi @vkucera Thank you very much for the clarification and guidance. I will follow the instructions in the documentation to properly clean up the include statements. Then I will update in the new PR once I have done it correctly.