[PWGLF] Fix clang-tidy findings in Omega2012 and K1 tables

Mark MC-only callback parameters as maybe_unused and remove redundant
integer casts. Bind fixed-size daughter index arrays by reference.
Keep the persistent K1 feature column format and document its narrow
exception to modernize-avoid-c-arrays.

Author: Bong-Hwi Lim

PWGLF/DataModel/LFK1MlTables.h

@@ -63,6 +63,7 @@ DECLARE_SOA_COLUMN(K1Charge, k1Charge, int8_t);
 // 16 = candidate cuts.
 // Candidates written at the "selected" export stage carry 31.
 DECLARE_SOA_COLUMN(K1BaselinePassBits, k1BaselinePassBits, uint16_t);
+// NOLINTNEXTLINE(modernize-avoid-c-arrays) -- persistent fixed-size AOD column
 DECLARE_SOA_COLUMN(K1MasterFeatures, k1MasterFeatures, float[125]); //! 125 master features of the K1 ML feature contract (PWGLF/Core/K1MlFeatures.h)
 DECLARE_SOA_COLUMN(K1FeatureStatus, k1FeatureStatus, uint8_t);      //! o2::analysis::k1ml::BuildStatus; only Ok (0) rows are written
 DECLARE_SOA_COLUMN(K1TruthStatus, k1TruthStatus, uint8_t);          //! 0 data, 1 matched, 2 unmatched

PWGLF/Tasks/Resonances/omega2012Analysis.cxx

@@ -698,7 +698,7 @@ struct Omega2012Analysis {
     };
 
     const bool fillWrongSign = core.fillWrongSign();
-    auto onCandidate = [&](auto const& xi, auto const& pion, auto const& kaon, Xi1530KCandidateValues const& c) {
+    auto onCandidate = [&]([[maybe_unused]] auto const& xi, [[maybe_unused]] auto const& pion, [[maybe_unused]] auto const& kaon, Xi1530KCandidateValues const& c) {
       if (c.chargePattern != o2::analysis::omega2012ml::kSignalPattern) {
         // Charge-pattern controls: same event only, never in the signal histograms
         if constexpr (!IsMix) {

PWGLF/Tasks/Resonances/omega2012TrainingTable.cxx

@@ -274,7 +274,7 @@ struct Omega2012TrainingTable {
       return -1;
     }
     const auto row = trackId - static_cast<int64_t>(mcTrackLabelsDf->offset());
-    if (row < 0 || row >= static_cast<int64_t>(mcTrackLabelsDf->size())) {
+    if (row < 0 || row >= mcTrackLabelsDf->size()) {
       return -1;
     }
     return mcTrackLabelsDf->rawIteratorAt(row).mcParticleId();
@@ -289,7 +289,7 @@ struct Omega2012TrainingTable {
     int64_t row = label;
     for (int step = 0; step <= MaxAncestorSteps; ++step) {
       const auto local = row - static_cast<int64_t>(mcParticlesDf->offset());
-      if (row < 0 || local < 0 || local >= static_cast<int64_t>(mcParticlesDf->size())) {
+      if (row < 0 || local < 0 || local >= mcParticlesDf->size()) {
         return;
       }
       const auto particle = mcParticlesDf->rawIteratorAt(local);
@@ -311,7 +311,7 @@ struct Omega2012TrainingTable {
     if (auto it = mlCascadeRows.find(id); it != mlCascadeRows.end()) {
       return it->second;
     }
-    const auto indices = xi.cascadeIndices();
+    const auto& indices = xi.cascadeIndices();
     const std::array<int, 3> daughterIds{indices[0], indices[1], indices[2]};
     mlCascades(mlEventRow, id, xi.px(), xi.py(), xi.pz(), static_cast<int8_t>(xi.sign()), xi.mXi(), xi.mLambda(),
                xi.v0CosPA(), xi.cascCosPA(), xi.daughDCA(), xi.cascDaughDCA(),
@@ -344,7 +344,7 @@ struct Omega2012TrainingTable {
     if (auto it = mlV0Rows.find(id); it != mlV0Rows.end()) {
       return it->second;
     }
-    const auto indices = v0.indices();
+    const auto& indices = v0.indices();
     const std::array<int, 2> daughterIds{indices[0], indices[1]};
     mlV0s(mlEventRow, id, v0.px(), v0.py(), v0.pz(), v0.mK0Short(), v0.mLambda(), v0.mAntiLambda(),
           v0.v0CosPA(), v0.daughDCA(), v0.dcapostopv(), v0.dcanegtopv(), v0.dcav0topv(), v0.transRadius(),
@@ -557,14 +557,14 @@ struct Omega2012TrainingTable {
   {
     mlAncestryRows.clear();
     const auto offset = static_cast<int64_t>(mcParticles.offset());
-    const auto nRows = static_cast<int64_t>(mcParticles.size());
+    const auto nRows = mcParticles.size();
     std::unordered_set<int64_t> treeRows;
     std::vector<std::pair<int64_t, int>> stack;
     for (const auto& parent : mcParticles) {
       if (std::abs(parent.pdgCode()) != o2::constants::physics::Pdg::kOmega2012Minus) {
         continue;
       }
-      stack.assign(1, {static_cast<int64_t>(parent.globalIndex()), 0});
+      stack.assign(1, {parent.globalIndex(), 0});
       while (!stack.empty()) {
         const auto [row, depth] = stack.back();
         stack.pop_back();