HBFUtils.nHBFperTF is stored in GRP, re-set default to 256 (#5901)

* HBFUtils.nHBFperTF is stored in GRP, re-set default to 256

* dummy commit to trigger CI

Co-authored-by: shahoian <ruben.shahoyan@cern.ch>

Author: shahor02

DataFormats/Parameters/include/DataFormatsParameters/GRPObject.h

@@ -53,6 +53,9 @@ class GRPObject
   void setTimeStart(timePoint t) { mTimeStart = t; }
   void setTimeEnd(timePoint t) { mTimeEnd = t; }
 
+  void setNHBFPerTF(uint32_t n) { mNHBFPerTF = n; }
+  uint32_t getNHBFPerTF() const { return mNHBFPerTF; }
+
   void setFirstOrbit(uint32_t o) { mFirstOrbit = o; }
   uint32_t getFirstOrbit() const { return mFirstOrbit; }
 
@@ -138,10 +141,11 @@ class GRPObject
   static GRPObject* loadFrom(const std::string& grpFileName, const std::string& grpName = "GRP");
 
  private:
-  timePoint mTimeStart = 0; ///< DAQ_time_start entry from DAQ logbook
-  timePoint mTimeEnd = 0;   ///< DAQ_time_end entry from DAQ logbook
+  timePoint mTimeStart = 0;      ///< DAQ_time_start entry from DAQ logbook
+  timePoint mTimeEnd = LONG_MAX; ///< DAQ_time_end entry from DAQ logbook
 
-  uint32_t mFirstOrbit = 0; /// 1st orbit of the 1st TF, in the MC set at digitization // RS Not sure it will stay in GRP, may go to some CTP object
+  uint32_t mFirstOrbit = 0;  /// 1st orbit of the 1st TF, in the MC set at digitization // RS Not sure it will stay in GRP, may go to some CTP object
+  uint32_t mNHBFPerTF = 256; /// Number of HBFrames per TF
 
   DetID::mask_t mDetsReadout;      ///< mask of detectors which are read out
   DetID::mask_t mDetsContinuousRO; ///< mask of detectors read out in continuos mode

DataFormats/Parameters/src/GRPObject.cxx

@@ -50,7 +50,7 @@ void GRPObject::print() const
   printf("Start: %s", std::ctime(&t));
   t = mTimeEnd; // system_clock::to_time_t(mTimeEnd);
   printf("End  : %s", std::ctime(&t));
-  printf("1st orbit: %u\n", mFirstOrbit);
+  printf("1st orbit: %u, %u orbits per TF\n", mFirstOrbit, mNHBFPerTF);
   printf("Beam0: Z:A = %3d:%3d, Energy = %.3f\n", getBeamZ(BeamClockWise), getBeamA(BeamClockWise),
          getBeamEnergyPerNucleon(BeamClockWise));
   printf("Beam1: Z:A = %3d:%3d, Energy = %.3f\n", getBeamZ(BeamAntiClockWise), getBeamA(BeamAntiClockWise),

Detectors/Raw/include/DetectorsRaw/HBFUtils.h

@@ -125,7 +125,7 @@ struct HBFUtils : public o2::conf::ConfigurableParamHelper<HBFUtils> {
 
   void print() const { printKeyValues(true); }
 
-  int nHBFPerTF = 128;     ///< number of orbits per BC
+  int nHBFPerTF = 256;     ///< number of orbits per BC
   uint32_t orbitFirst = 0; ///< orbit of 1st TF of the run
 
   // used for MC

Steer/DigitizerWorkflow/src/GRPUpdaterSpec.cxx

@@ -52,7 +52,7 @@ class GRPDPLUpdatedTask
 
     TFile flGRP(mGRPFileName.c_str(), "update");
     if (flGRP.IsZombie()) {
-      LOG(ERROR) << "Failed to open  in update mode " << mGRPFileName;
+      LOG(ERROR) << "Failed to open in update mode " << mGRPFileName;
       return;
     }
     std::unique_ptr<GRP> grp(static_cast<GRP*>(flGRP.GetObjectChecked(grpName.c_str(), GRP::Class())));
@@ -65,6 +65,7 @@ class GRPDPLUpdatedTask
       grp->setDetROMode(det, roMode);
     }
     grp->setFirstOrbit(o2::raw::HBFUtils::Instance().orbitFirst);
+    grp->setNHBFPerTF(o2::raw::HBFUtils::Instance().nHBFPerTF);
     LOG(INFO) << "Updated GRP in " << mGRPFileName << " for detectors RO mode and 1st orbit of the run";
     grp->print();
     flGRP.WriteObjectAny(grp.get(), grp->Class(), grpName.c_str());