[PWGLF] Add event selection flag to the he3hadmult table

[GiorgioAlbertoLucia]

No description provided.

[github-actions]

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

[alibuild]

Error while checking build/O2Physics/staging for 036f374 at 2026-08-05 09:07:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

[alibuild]

Error while checking build/O2Physics/code-check for 036f374 at 2026-08-08 03:01:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 34 errors and 4 warnings.'
Found 34 errors and 4 warnings.
++ [[ 34 -gt 0 ]]
++ cat /sw/BUILD/eb3ef1afef2beb02d730fa16d84c7fed86063953/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:251:8: error: constructor does not initialize these fields: mCcdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:288:85: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:573:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:625:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:677:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:189:12: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:194:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:200:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:206:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:212:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:218:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:224:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:230:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:236:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:242:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:248:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:254:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:260:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:267:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:268:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:269:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:270:18: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:283:1: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:283:7: error: variable 'pidCutTPC' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:285:27: error: variable 'lut' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:285:27: error: variable 'lut' provides global access to a non-const object; consider making the pointed-to data 'const' [cppcoreguidelines-avoid-non-const-global-variables,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:287:31: error: variable 'candidates' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:288:35: error: variable 'candidates_flow' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:319:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:333:127: error: the parameter 'eventSelections' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:394:102: error: the parameter 'hFailCentrality' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:407:5: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:613:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Utils/nucleiUtils.h:616:3: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
++ [[ 4 -gt 0 ]]
++ cat /sw/BUILD/eb3ef1afef2beb02d730fa16d84c7fed86063953/O2Physics-code-check/warnings.txt
PWGLF/Utils/nucleiUtils.h:495:25: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Utils/nucleiUtils.h:504:25: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Utils/nucleiUtils.h:514:25: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Utils/nucleiUtils.h:567:35: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 34 -gt 0 ]]
++ exit 1
[0 more errors; see full log]

Full log here.

[alibuild]

Error while checking build/O2Physics/o2 for 036f374 at 2026-08-05 10:03:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

[vkucera]

@GiorgioAlbertoLucia Please fix the PR title.

It's is wrong and very confusing. It's a statement and it's not clear whether it's the reason of the PR or its outcome. Plus "now'" is meaningless in this context. It doesn't convey any information about the scope of the changes either. Please read the references in the contribution guidelines and learn to formulate a proper commit message.

[GiorgioAlbertoLucia]

@GiorgioAlbertoLucia Please fix the PR title.

It's is wrong and very confusing. It's a statement and it's not clear whether it's the reason of the PR or its outcome. Plus "now'" is meaningless in this context. It doesn't convey any information about the scope of the changes either. Please read the references in the contribution guidelines and learn to formulate a proper commit message.

@vkucera the PR title has been fixed. I removed the "now" and it is now not a statement, but the content is the same. The only goal of the pr is to store event selection flags in the output table, specifically, in the multiplicity table. If you are still not convinced, please feel free to leave a suggestion

[vkucera]

Thanks @GiorgioAlbertoLucia , but the title is still wrong and still does not follow the guidelines. The multiplicity table is produced by Common/TableProducer/multCentTable.cxx. This PR is not touching it.

[GiorgioAlbertoLucia]

Thanks @GiorgioAlbertoLucia , but the title is still wrong and still does not follow the guidelines. The multiplicity table is produced by Common/TableProducer/multCentTable.cxx. This PR is not touching it.

@vkucera I understand the confusion. The title referred to the multiplicity table pertinent to the analysis task. The initial comment was unclear about what the issue was. Hopefully, now the title is clearer.

[vkucera]

Thanks @GiorgioAlbertoLucia , but the title is still wrong and still does not follow the guidelines. The multiplicity table is produced by Common/TableProducer/multCentTable.cxx. This PR is not touching it.

@vkucera I understand the confusion. The title referred to the multiplicity table pertinent to the analysis task. The initial comment was unclear about what the issue was. Hopefully, now the title is clearer.

Thanks. Please use the imperative mood in your future PRs (as instructed in the guidelines).