TRD: CalibratorVdExB: Clang-format on all touched files

This is just a minor patch on all files the previous patches touched.
Mostly comment alignment which IMHO are nicer.
Also some minor touch-ups on comments to allow doxygen to pickup as
proper documentation.

Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>

Author: f3sch

DataFormats/Detectors/TRD/include/DataFormatsTRD/Constants.h

@@ -22,75 +22,75 @@ namespace trd
 {
 namespace constants
 {
-constexpr int NSECTOR = 18;          // the number of sectors
-constexpr int NSTACK = 5;            // the number of stacks per sector
-constexpr int NLAYER = 6;            // the number of layers
-constexpr int NCHAMBERPERSEC = 30;   // the number of chambers per sector
-constexpr int NHCPERSEC = 60;        // the number of half-chambers per sector
-constexpr int MAXCHAMBER = 540;      // the maximum number of installed chambers
-constexpr int MAXHALFCHAMBER = 1080; // the maximum number of installed half-chambers
-constexpr int NCHAMBER = 521;        // the number of chambers actually installed
-constexpr int NHALFCRU = 72;         // the number of half cru (link bundles)
-constexpr int NLINKSPERHALFCRU = 15; // the number of links per half cru or cru end point.
-constexpr int NLINKSPERCRU = 30;     // the number of links per CRU (two CRUs serve one supermodule)
-constexpr int NCRU = 36;             // the number of CRU we have
-constexpr int NFLP = 12;             // the number of FLP we have.
-constexpr int NCRUPERFLP = 3;        // the number of CRU per FLP
-constexpr int TRDLINKID = 15;        // hard coded link id, specific to TRD
+constexpr int NSECTOR = 18;          ///< the number of sectors
+constexpr int NSTACK = 5;            ///< the number of stacks per sector
+constexpr int NLAYER = 6;            ///< the number of layers
+constexpr int NCHAMBERPERSEC = 30;   ///< the number of chambers per sector
+constexpr int NHCPERSEC = 60;        ///< the number of half-chambers per sector
+constexpr int MAXCHAMBER = 540;      ///< the maximum number of installed chambers
+constexpr int MAXHALFCHAMBER = 1080; ///< the maximum number of installed half-chambers
+constexpr int NCHAMBER = 521;        ///< the number of chambers actually installed
+constexpr int NHALFCRU = 72;         ///< the number of half cru (link bundles)
+constexpr int NLINKSPERHALFCRU = 15; ///< the number of links per half cru or cru end point.
+constexpr int NLINKSPERCRU = 30;     ///< the number of links per CRU (two CRUs serve one supermodule)
+constexpr int NCRU = 36;             ///< the number of CRU we have
+constexpr int NFLP = 12;             ///< the number of FLP we have.
+constexpr int NCRUPERFLP = 3;        ///< the number of CRU per FLP
+constexpr int TRDLINKID = 15;        ///< hard coded link id, specific to TRD
 
-constexpr int NCOLUMN = 144; // the number of pad columns for each chamber
-constexpr int NROWC0 = 12;   // the number of pad rows for chambers of type C0 (installed stack 0,1,3 and 4)
-constexpr int NROWC1 = 16;   // the number of pad rows for chambers of type C1 (installed in stack 2)
+constexpr int NCOLUMN = 144; ///< the number of pad columns for each chamber
+constexpr int NROWC0 = 12;   ///< the number of pad rows for chambers of type C0 (installed stack 0,1,3 and 4)
+constexpr int NROWC1 = 16;   ///< the number of pad rows for chambers of type C1 (installed in stack 2)
 
-constexpr int NMCMROB = 16;     // the number of MCMs per ROB
-constexpr int NMCMHCMAX = 64;   // the maximum number of MCMs for one half chamber (C1 type)
-constexpr int NMCMROBINROW = 4; // the number of MCMs per ROB in row direction
-constexpr int NMCMROBINCOL = 4; // the number of MCMs per ROB in column direction
-constexpr int NROBC0 = 6;       // the number of ROBs per C0 chamber
-constexpr int NROBC1 = 8;       // the number of ROBs per C1 chamber
-constexpr int NADCMCM = 21;     // the number of ADC channels per MCM
-constexpr int NCOLMCM = 18;     // the number of pads per MCM
-constexpr int NCPU = 4;         // the number of CPUs inside the TRAP chip
-constexpr int NCHARGES = 3;     // the number of charges per tracklet (Q0/1/2)
+constexpr int NMCMROB = 16;     ///< the number of MCMs per ROB
+constexpr int NMCMHCMAX = 64;   ///< the maximum number of MCMs for one half chamber (C1 type)
+constexpr int NMCMROBINROW = 4; ///< the number of MCMs per ROB in row direction
+constexpr int NMCMROBINCOL = 4; ///< the number of MCMs per ROB in column direction
+constexpr int NROBC0 = 6;       ///< the number of ROBs per C0 chamber
+constexpr int NROBC1 = 8;       ///< the number of ROBs per C1 chamber
+constexpr int NADCMCM = 21;     ///< the number of ADC channels per MCM
+constexpr int NCOLMCM = 18;     ///< the number of pads per MCM
+constexpr int NCPU = 4;         ///< the number of CPUs inside the TRAP chip
+constexpr int NCHARGES = 3;     ///< the number of charges per tracklet (Q0/1/2)
 
 // the values below should come out of the TRAP config in the future
-constexpr int NBITSTRKLPOS = 11;                                      // number of bits for position in tracklet64 word
-constexpr int NBITSTRKLSLOPE = 8;                                     // number of bits for slope in tracklet64 word
-constexpr int ADDBITSHIFTSLOPE = 1 << 3;                              // in the TRAP the slope is shifted by 3 additional bits compared to the position
-constexpr int PADGRANULARITYTRKLPOS = 40;                             // tracklet position is stored in units of 1/40 pad
-constexpr int PADGRANULARITYTRKLSLOPE = 128;                          // tracklet deflection is stored in units of 1/128 pad per time bin
-constexpr float GRANULARITYTRKLPOS = 1.f / PADGRANULARITYTRKLPOS;     // granularity of position in tracklet64 word in pad-widths
-constexpr float GRANULARITYTRKLSLOPE = 1.f / PADGRANULARITYTRKLSLOPE; // granularity of slope in tracklet64 word in pads/timebin
-constexpr int ADCBASELINE = 10;                                       // baseline in ADC units
+constexpr int NBITSTRKLPOS = 11;                                      ///< number of bits for position in tracklet64 word
+constexpr int NBITSTRKLSLOPE = 8;                                     ///< number of bits for slope in tracklet64 word
+constexpr int ADDBITSHIFTSLOPE = 1 << 3;                              ///< in the TRAP the slope is shifted by 3 additional bits compared to the position
+constexpr int PADGRANULARITYTRKLPOS = 40;                             ///< tracklet position is stored in units of 1/40 pad
+constexpr int PADGRANULARITYTRKLSLOPE = 128;                          ///< tracklet deflection is stored in units of 1/128 pad per time bin
+constexpr float GRANULARITYTRKLPOS = 1.f / PADGRANULARITYTRKLPOS;     ///< granularity of position in tracklet64 word in pad-widths
+constexpr float GRANULARITYTRKLSLOPE = 1.f / PADGRANULARITYTRKLSLOPE; ///< granularity of slope in tracklet64 word in pads/timebin
+constexpr int ADCBASELINE = 10;                                       ///< baseline in ADC units
 
 // OS: Should this not be flexible for example in case of Kr calib?
-constexpr int TIMEBINS = 30;           // the number of time bins
-constexpr float MAXIMPACTANGLE = 25.f; // the maximum impact angle for tracks relative to the TRD detector plane to be considered for vDrift and ExB calibration
-constexpr int NBINSANGLEDIFF = 25;     // the number of bins for the track angle used for the vDrift and ExB calibration based on the tracking
-constexpr double ANODEPLANE = 0.0335; // distance of the TRD anode plane from the drift cathodes in [m]
+constexpr int TIMEBINS = 30;           ///< the number of time bins
+constexpr float MAXIMPACTANGLE = 25.f; ///< the maximum impact angle for tracks relative to the TRD detector plane to be considered for vDrift and ExB calibration
+constexpr int NBINSANGLEDIFF = 25;     ///< the number of bins for the track angle used for the vDrift and ExB calibration based on the tracking
+constexpr double ANODEPLANE = 0.0335;  ///< distance of the TRD anode plane from the drift cathodes in [m]
 
 // Trigger parameters
-constexpr double READOUT_TIME = 3000;                  // the time the readout takes, as 30 TB = 3 micro-s.
-constexpr double DEAD_TIME = 200;                      // trigger deadtime, 2 micro-s
-constexpr double BUSY_TIME = READOUT_TIME + DEAD_TIME; // the time for which no new trigger can be received in nanoseconds
+constexpr double READOUT_TIME = 3000;                  ///< the time the readout takes, as 30 TB = 3 micro-s.
+constexpr double DEAD_TIME = 200;                      ///< trigger deadtime, 2 micro-s
+constexpr double BUSY_TIME = READOUT_TIME + DEAD_TIME; ///< the time for which no new trigger can be received in nanoseconds
 
 // array size to store incoming half cru payload.
-constexpr int HBFBUFFERMAX = 1048576;          // max buffer size for data read from a half cru, (all events)
-constexpr unsigned int CRUPADDING32 = 0xeeeeeeee; // padding word used in the cru.
-constexpr int CHANNELNRNOTRKLT = 23;           // this marks channels in the ADC mask which don't contribute to a tracklet
-constexpr int NOTRACKLETFIT = 31;              // this value is assigned to the fit pointer in case no tracklet is available
-constexpr int TRACKLETENDMARKER = 0x10001000;  // marker for the end of tracklets in raw data, 2 of these.
-constexpr int PADDINGWORD = 0xeeeeeeee;        // half-CRU links will be padded with this words to get an even number of 256bit words
-constexpr int DIGITENDMARKER = 0x0;            // marker for the end of digits in raw data, 2 of these
-constexpr int MAXDATAPERLINK32 = 13824;        // max number of 32 bit words per link ((21x12+2+4)*64) 64 mcm, 21 channels, 10 words per channel 2 header words(DigitMCMHeader DigitMCMADCmask) 4 words for tracklets.
-constexpr int MAXDATAPERLINK256 = 1728;        // max number of linkwords per cru link. (256bit words)
-constexpr int MAXEVENTCOUNTERSEPERATION = 200; // how far apart can subsequent mcmheader event counters be before we flag for concern, used as a sanity check in rawreader.
-constexpr int MAXMCMCOUNT = 69120;             // at most mcm count maxchamber x nrobc1 nmcmrob
-constexpr int MAXLINKERRORHISTOGRAMS = 10;     // size of the array holding the link error plots from the raw reader
-constexpr int MAXPARSEERRORHISTOGRAMS = 60;    // size of the array holding the parsing error plots from the raw reader
-constexpr unsigned int ETYPEPHYSICSTRIGGER = 0x2;     // CRU Half Chamber header eventtype definition
-constexpr unsigned int ETYPECALIBRATIONTRIGGER = 0x3; // CRU Half Chamber header eventtype definition
-constexpr int MAXCRUERRORVALUE = 0x2;          // Max possible value for a CRU Halfchamber link error. As of may 2022, can only be 0x0, 0x1, and 0x2, at least that is all so far(may2022).
+constexpr int HBFBUFFERMAX = 1048576;                 ///< max buffer size for data read from a half cru, (all events)
+constexpr unsigned int CRUPADDING32 = 0xeeeeeeee;     ///< padding word used in the cru.
+constexpr int CHANNELNRNOTRKLT = 23;                  ///< this marks channels in the ADC mask which don't contribute to a tracklet
+constexpr int NOTRACKLETFIT = 31;                     ///< this value is assigned to the fit pointer in case no tracklet is available
+constexpr int TRACKLETENDMARKER = 0x10001000;         ///< marker for the end of tracklets in raw data, 2 of these.
+constexpr int PADDINGWORD = 0xeeeeeeee;               ///< half-CRU links will be padded with this words to get an even number of 256bit words
+constexpr int DIGITENDMARKER = 0x0;                   ///< marker for the end of digits in raw data, 2 of these
+constexpr int MAXDATAPERLINK32 = 13824;               ///< max number of 32 bit words per link ((21x12+2+4)*64) 64 mcm, 21 channels, 10 words per channel 2 header words(DigitMCMHeader DigitMCMADCmask) 4 words for tracklets.
+constexpr int MAXDATAPERLINK256 = 1728;               ///< max number of linkwords per cru link. (256bit words)
+constexpr int MAXEVENTCOUNTERSEPERATION = 200;        ///< how far apart can subsequent mcmheader event counters be before we flag for concern, used as a sanity check in rawreader.
+constexpr int MAXMCMCOUNT = 69120;                    ///< at most mcm count maxchamber x nrobc1 nmcmrob
+constexpr int MAXLINKERRORHISTOGRAMS = 10;            ///< size of the array holding the link error plots from the raw reader
+constexpr int MAXPARSEERRORHISTOGRAMS = 60;           ///< size of the array holding the parsing error plots from the raw reader
+constexpr unsigned int ETYPEPHYSICSTRIGGER = 0x2;     ///< CRU Half Chamber header eventtype definition
+constexpr unsigned int ETYPECALIBRATIONTRIGGER = 0x3; ///< CRU Half Chamber header eventtype definition
+constexpr int MAXCRUERRORVALUE = 0x2;                 ///< Max possible value for a CRU Halfchamber link error. As of may 2022, can only be 0x0, 0x1, and 0x2, at least that is all so far(may2022).
 
 } // namespace constants
 } // namespace trd

Detectors/TRD/calibration/include/TRDCalibration/CalibratorVdExB.h

@@ -38,8 +38,8 @@ struct FitFunctor {
   double operator()(const double* par) const;
   double calculateDeltaAlphaSim(double vdFit, double laFit, double impactAng) const;
   std::array<std::unique_ptr<TProfile>, constants::MAXCHAMBER> profiles; ///< profile histograms for each TRD chamber
-  std::array<double,constants::MAXCHAMBER> vdPreCorr;                                                      ///< vDrift from previous Run
-  std::array<double,constants::MAXCHAMBER> laPreCorr;                                                      ///< LorentzAngle from previous Run
+  std::array<double, constants::MAXCHAMBER> vdPreCorr;                   ///< vDrift from previous Run
+  std::array<double, constants::MAXCHAMBER> laPreCorr;                   ///< LorentzAngle from previous Run
   int currDet;                                                           ///< the current TRD chamber number
   float lowerBoundAngleFit;
   float upperBoundAngleFit;
@@ -72,11 +72,11 @@ class CalibratorVdExB final : public o2::calibration::TimeSlotCalibration<o2::tr
   void retrievePrev(o2::framework::ProcessingContext& pc);
 
  private:
-  bool mInitDone{false}; ///< flag to avoid creating the TProfiles multiple times
-  size_t mMinEntriesTotal; ///< minimum total number of angular deviations (on average ~3 entries per bin for each TRD chamber)
-  size_t mMinEntriesChamber; ///< minimum number of angular deviations per chamber for accepting refitted value (~3 per bin)
-  bool mEnableOutput; //< enable output of calibration fits and tprofiles in a root file instead of the ccdb
-  FitFunctor mFitFunctor; ///< used for minimization procedure
+  bool mInitDone{false};                             ///< flag to avoid creating the TProfiles multiple times
+  size_t mMinEntriesTotal;                           ///< minimum total number of angular deviations (on average ~3 entries per bin for each TRD chamber)
+  size_t mMinEntriesChamber;                         ///< minimum number of angular deviations per chamber for accepting refitted value (~3 per bin)
+  bool mEnableOutput;                                //< enable output of calibration fits and tprofiles in a root file instead of the ccdb
+  FitFunctor mFitFunctor;                            ///< used for minimization procedure
   std::vector<o2::ccdb::CcdbObjectInfo> mInfoVector; ///< vector of CCDB infos; each element is filled with CCDB description of accompanying CCDB calibration object
   std::vector<o2::trd::CalVdriftExB> mObjectVector;  ///< vector of CCDB calibration objects; the extracted vDrift and ExB per chamber for given slot
   ClassDefOverride(CalibratorVdExB, 3);

Detectors/TRD/calibration/src/CalibratorVdExB.cxx

@@ -161,7 +161,7 @@ void CalibratorVdExB::finalizeSlot(Slot& slot)
       }
     }
     // Check if we have the minimum amount of entries
-    if(sumEntries < mMinEntriesChamber){
+    if (sumEntries < mMinEntriesChamber) {
       LOGF(debug, "Chamber %d did not reach minimum amount of entries for refit", iDet);
       continue;
     }

Detectors/TRD/workflow/include/TRDWorkflow/VdAndExBCalibSpec.h

@@ -148,7 +148,6 @@ DataProcessorSpec getTRDVdAndExBCalibSpec()
       {"enable-root-output", VariantType::Bool, false, {"output tprofiles and fits to root file"}},
       {"min-entries-chamber", VariantType::Int, 75, {"minimum number of entries per chamber to fit single time slot"}},
       {"min-entries-total", VariantType::Int, 40'500, {"total minimum number of entries to fit single time slot"}}}}; // around 3 entries per bin per chamber
-
 }
 
 } // namespace framework