Fix clang-format indentation, cppcheck const-array warning, duplicate README heading

Author: forynski

Tools/PIDFeatureExtractor/README.md

@@ -33,7 +33,7 @@ cuts, writes one row containing:
 Mode is a runtime switch - enable `processData` for real data or
 `processMc` for MC (reconstructed + truth), not both.
 
-### Configurable options
+### PidFeatureExtractor options
 
 | Option | Default | What it does |
 |---|---|---|
@@ -71,7 +71,7 @@ detector configuration - each group can be switched off independently;
 turning one off overrides the data for that group, the same way a genuine
 detector miss would look.
 
-### Configurable options
+### PidOnnxInference options
 
 | Option | Default | What it does |
 |---|---|---|

Tools/PIDFeatureExtractor/pidFeatureExtractor.cxx

@@ -296,7 +296,7 @@ struct PidFeatureExtractor {
   /// Bayesian PID: requires TPC (a TPC-only track still gets a real
   /// posterior); folds in TOF too when also present. NaN in all four
   /// outputs if TPC is absent or computeBayesianPid is false.
-  void computeBayesianProbs(bool hasTpcIn, float nsTPC[4], bool hasTofIn, float nsTOF[4], float out[4]) const
+  void computeBayesianProbs(bool hasTpcIn, const float nsTPC[4], bool hasTofIn, const float nsTOF[4], float out[4]) const
   {
     if (!computeBayesianPid.value || !hasTpcIn) {
       out[0] = out[1] = out[2] = out[3] = kNaN;

Tools/PIDFeatureExtractor/pidOnnxInference.cxx

@@ -115,8 +115,8 @@ struct GroupToggles {
 /// (assumed always-present in the data itself), so their toggle is the
 /// only way to exclude them.
 void runInference(std::string const& inputRootFile, std::string const& inputTreeName,
-                   std::string const& outputPath, bool exportCsv,
-                   o2::analysis::MlResponse<float>& mlResponse, GroupToggles const& groups)
+                  std::string const& outputPath, bool exportCsv,
+                  o2::analysis::MlResponse<float>& mlResponse, GroupToggles const& groups)
 {
   std::unique_ptr<TFile> inFile(TFile::Open(inputRootFile.c_str(), "READ"));
   if (!inFile || inFile->IsZombie()) {