Please consider the following formatting changes

Author: alibuild

DataFormats/Detectors/TPC/include/DataFormatsTPC/CMV.h

@@ -25,56 +25,57 @@
 namespace o2::tpc::cmv
 {
 
-static constexpr uint32_t NTimeBins = 3564;                                     ///< number of time bins (spans 8 orbits)
-static constexpr uint32_t SignificantBits = 2;                                  ///< number of bits used for floating point precision
-static constexpr float FloatConversion = 1.f / float(1 << SignificantBits);     ///< conversion factor from integer representation to float
+static constexpr uint32_t NTimeBins = 3564;                                 ///< number of time bins (spans 8 orbits)
+static constexpr uint32_t SignificantBits = 2;                              ///< number of bits used for floating point precision
+static constexpr float FloatConversion = 1.f / float(1 << SignificantBits); ///< conversion factor from integer representation to float
 
 /// Header definition of the CMVs
 struct Header {
   static constexpr uint8_t MagicWord = 0xDC;
   union {
-    uint32_t word0 = 0;              ///< bits 0 - 31
+    uint32_t word0 = 0; ///< bits 0 - 31
     struct {
-      uint8_t version : 8;           ///< version
-      uint8_t packetID : 8;          ///< packet id
-      uint8_t errorCode : 8;         ///< errors
-      uint8_t magicWord : 8;         ///< magic word
+      uint8_t version : 8;   ///< version
+      uint8_t packetID : 8;  ///< packet id
+      uint8_t errorCode : 8; ///< errors
+      uint8_t magicWord : 8; ///< magic word
     };
   };
   union {
-    uint32_t word1 = 0;              ///< bits 32 - 63
+    uint32_t word1 = 0; ///< bits 32 - 63
     struct {
-      uint32_t heartbeatOrbit : 32;  ///< first heart beat timing of the package 
+      uint32_t heartbeatOrbit : 32; ///< first heart beat timing of the package
     };
   };
   union {
-    uint16_t word2 = 0;              ///< bits 64 - 79
+    uint16_t word2 = 0; ///< bits 64 - 79
     struct {
-      uint16_t heartbeatBC : 16;     ///< first BC id of the package
+      uint16_t heartbeatBC : 16; ///< first BC id of the package
     };
   };
 };
 
 /// CMV single data container
 struct Data {
-  uint16_t CMV{0};                   ///< 16bit ADC value
-  
+  uint16_t CMV{0}; ///< 16bit ADC value
+
   // Raw integer accessors
   uint16_t getCMV() const { return CMV; }
   void setCMV(uint16_t value) { CMV = value; }
-  
+
   // Float helpers using SignificantBits for fixed-point conversion
   float getCMVFloat() const { return static_cast<float>(CMV) * FloatConversion; }
-  void setCMVFloat(float value) {
+  void setCMVFloat(float value)
+  {
     // round to nearest representable fixed-point value
     setCMV(uint32_t((value + 0.5f * FloatConversion) / FloatConversion));
   }
 };
 
 /// CMV full data container: one packet carries NTimeBins time bins
 struct Container {
-  Header header;                  ///< CMV data header
-  Data data[NTimeBins];           ///< data values for given number of time bins
+  Header header;        ///< CMV data header
+  Data data[NTimeBins]; ///< data values for given number of time bins
 
   // Header and data accessors
   const Header& getHeader() const { return header; }
@@ -89,7 +90,6 @@ struct Container {
 
   float getCMVFloat(uint32_t timeBin) const { return data[timeBin].getCMVFloat(); }
   void setCMVFloat(uint32_t timeBin, float value) { data[timeBin].setCMVFloat(value); }
-
 };
 } // namespace o2::tpc::cmv
 

Detectors/TPC/workflow/include/TPCWorkflow/CMVToVectorSpec.h

@@ -28,4 +28,4 @@ o2::framework::DataProcessorSpec getCMVToVectorSpec(const std::string inputSpec,
 
 } // end namespace o2::tpc
 
-#endif //TPC_CMVToVectorSpec_H_
+#endif // TPC_CMVToVectorSpec_H_

Detectors/TPC/workflow/src/CMVToVectorSpec.cxx

@@ -263,17 +263,17 @@ class CMVToVectorDevice : public o2::framework::Task
     bool matches(uint32_t orbit, int16_t bc) const { return ((heartbeatOrbit == orbit) && (heartbeatBC == bc)); }
   };
 
-  int mRawDataType{0};                                                                          ///< type of raw data to dump in case of errors
-  bool mWriteDebug{false};                                                                      ///< write a debug output
-  bool mWriteDebugOnError{false};                                                               ///< write a debug output in case of errors
-  bool mWriteRawDataOnError{false};                                                             ///< write raw data in case of errors
-  std::vector<uint32_t> mCRUs;                                                                  ///< CRUs expected for this device
-  std::unordered_map<uint32_t, std::vector<float>> mCMVvectors;                                 ///< decoded CMVs per cru over all CMV packets in the TF
-  std::unordered_map<uint32_t, std::vector<CMVInfo>> mCMVInfos;                                 ///< CMV packet information within the TF
-  std::string mDebugStreamFileName;                                                             ///< name of the debug stream output file
-  std::unique_ptr<o2::utils::TreeStreamRedirector> mDebugStream;                                ///< debug output streamer
-  std::ofstream mRawOutputFile;                                                                 ///< raw output file
-  std::string mRawOutputFileName;                                                               ///< name of the raw output file
+  int mRawDataType{0};                                           ///< type of raw data to dump in case of errors
+  bool mWriteDebug{false};                                       ///< write a debug output
+  bool mWriteDebugOnError{false};                                ///< write a debug output in case of errors
+  bool mWriteRawDataOnError{false};                              ///< write raw data in case of errors
+  std::vector<uint32_t> mCRUs;                                   ///< CRUs expected for this device
+  std::unordered_map<uint32_t, std::vector<float>> mCMVvectors;  ///< decoded CMVs per cru over all CMV packets in the TF
+  std::unordered_map<uint32_t, std::vector<CMVInfo>> mCMVInfos;  ///< CMV packet information within the TF
+  std::string mDebugStreamFileName;                              ///< name of the debug stream output file
+  std::unique_ptr<o2::utils::TreeStreamRedirector> mDebugStream; ///< debug output streamer
+  std::ofstream mRawOutputFile;                                  ///< raw output file
+  std::string mRawOutputFileName;                                ///< name of the raw output file
 
   //____________________________________________________________________________
   bool snapshotCMVs(DataAllocator& output, uint32_t tfCounter)
@@ -336,17 +336,16 @@ class CMVToVectorDevice : public o2::framework::Task
       if (mCMVInfos.find(cru) == mCMVInfos.end()) {
         continue;
       }
-    
+
       auto& infos = mCMVInfos[cru];
       auto& cmvVec = mCMVvectors[cru];
-    
+
       stream << "cru=" << cru
              << "tfCounter=" << tfCounter
              << "nCMVs=" << cmvVec.size()
              << "cmvs=" << cmvVec
              << "\n";
     }
-    
   }
 
   void writeRawData(InputRecord& inputs)