[O2-2538][FV0] Reduce excessive log printout of FV0 digitizer

Maciej Slupecki · shahor02 · commit af3e5d5c155e · 2021-08-29T22:11:01.000+02:00
diff --git a/Detectors/FIT/FV0/simulation/src/Digitizer.cxx b/Detectors/FIT/FV0/simulation/src/Digitizer.cxx
@@ -37,7 +37,7 @@ void Digitizer::clear()
 //_______________________________________________________________________
 void Digitizer::init()
 {
-  LOG(INFO) << "V0Digitizer::init -> start = ";
+  LOG(INFO) << "init";
   mNBins = FV0DigParam::Instance().waveformNbins;      //Will be computed using detector set-up from CDB
   mBinSize = FV0DigParam::Instance().waveformBinWidth; //Will be set-up from CDB
   mNTimeBinsPerBC = std::lround(o2::constants::lhc::LHCBunchSpacingNS / mBinSize); // 1920 bins/BC
@@ -77,7 +77,7 @@ void Digitizer::init()
   }
   mLastBCCache.clear();
   mCfdStartIndex.fill(0);
-  LOG(INFO) << "V0Digitizer::init -> finished";
+  LOG(INFO) << "init -> finished";
 }
 
 void Digitizer::process(const std::vector<o2::fv0::Hit>& hits,
@@ -86,7 +86,7 @@ void Digitizer::process(const std::vector<o2::fv0::Hit>& hits,
                         std::vector<o2::fv0::DetTrigInput>& digitsTrig,
                         o2::dataformats::MCTruthContainer<o2::fv0::MCLabel>& labels)
 {
-  LOG(INFO) << "[FV0] Digitizer::process(): begin with " << hits.size() << " hits";
+  LOG(DEBUG) << "Begin with " << hits.size() << " hits";
   flush(digitsBC, digitsCh, digitsTrig, labels); // flush cached signal which cannot be affect by new event
 
   std::vector<int> hitIdx(hits.size());
diff --git a/Steer/DigitizerWorkflow/src/FV0DigitizerSpec.cxx b/Steer/DigitizerWorkflow/src/FV0DigitizerSpec.cxx
@@ -49,7 +49,7 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer
 
   void initDigitizerTask(framework::InitContext& ic) override
   {
-    LOG(INFO) << "FV0DPLDigitizerTask:init";
+    LOG(DEBUG) << "FV0DPLDigitizerTask:init";
     mDigitizer.init();
     mDisableQED = ic.options().get<bool>("disable-qed"); //TODO: QED implementation to be tested
   }
@@ -59,7 +59,7 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer
     if (mFinished) {
       return;
     }
-    LOG(INFO) << "FV0DPLDigitizerTask:run";
+    LOG(DEBUG) << "FV0DPLDigitizerTask:run";
 
     // read collision context from input
     auto context = pc.inputs().get<o2::steer::DigitizationContext*>("collisioncontext");
@@ -83,14 +83,14 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer
       for (auto& part : eventParts[collID]) {
         hits.clear();
         context->retrieveHits(mSimChains, "FV0Hit", part.sourceID, part.entryID, &hits);
-        LOG(INFO) << "[FV0] For collision " << collID << " eventID " << part.entryID << " found " << hits.size() << " hits ";
+        LOG(DEBUG) << "[FV0] For collision " << collID << " eventID " << part.entryID << " found " << hits.size() << " hits ";
 
         // call actual digitization procedure
         mDigitizer.setEventId(part.entryID);
         mDigitizer.setSrcId(part.sourceID);
         mDigitizer.process(hits, mDigitsBC, mDigitsCh, mDigitsTrig, mLabels);
       }
-      LOG(INFO) << "[FV0] Has " << mDigitsBC.size() << " BC elements,   " << mDigitsCh.size() << " mDigitsCh elements";
+      LOG(DEBUG) << "[FV0] Has " << mDigitsBC.size() << " BC elements,   " << mDigitsCh.size() << " mDigitsCh elements";
     }
 
     o2::InteractionTimeRecord terminateIR;

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ void Digitizer::clear()`
`37`	`37`	`//_______________________________________________________________________`
`38`	`38`	`void Digitizer::init()`
`39`	`39`	`{`
`40`		`- LOG(INFO) << "V0Digitizer::init -> start = ";`
	`40`	`+ LOG(INFO) << "init";`
`41`	`41`	`mNBins = FV0DigParam::Instance().waveformNbins; //Will be computed using detector set-up from CDB`
`42`	`42`	`mBinSize = FV0DigParam::Instance().waveformBinWidth; //Will be set-up from CDB`
`43`	`43`	`mNTimeBinsPerBC = std::lround(o2::constants::lhc::LHCBunchSpacingNS / mBinSize); // 1920 bins/BC`
`@@ -77,7 +77,7 @@ void Digitizer::init()`
`77`	`77`	`}`
`78`	`78`	`mLastBCCache.clear();`
`79`	`79`	`mCfdStartIndex.fill(0);`
`80`		`- LOG(INFO) << "V0Digitizer::init -> finished";`
	`80`	`+ LOG(INFO) << "init -> finished";`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`void Digitizer::process(const std::vector<o2::fv0::Hit>& hits,`
`@@ -86,7 +86,7 @@ void Digitizer::process(const std::vector<o2::fv0::Hit>& hits,`
`86`	`86`	`std::vector<o2::fv0::DetTrigInput>& digitsTrig,`
`87`	`87`	`o2::dataformats::MCTruthContainer<o2::fv0::MCLabel>& labels)`
`88`	`88`	`{`
`89`		`- LOG(INFO) << "[FV0] Digitizer::process(): begin with " << hits.size() << " hits";`
	`89`	`+ LOG(DEBUG) << "Begin with " << hits.size() << " hits";`
`90`	`90`	`flush(digitsBC, digitsCh, digitsTrig, labels); // flush cached signal which cannot be affect by new event`
`91`	`91`
`92`	`92`	`std::vector<int> hitIdx(hits.size());`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer`
`49`	`49`
`50`	`50`	`void initDigitizerTask(framework::InitContext& ic) override`
`51`	`51`	`{`
`52`		`- LOG(INFO) << "FV0DPLDigitizerTask:init";`
	`52`	`+ LOG(DEBUG) << "FV0DPLDigitizerTask:init";`
`53`	`53`	`mDigitizer.init();`
`54`	`54`	`mDisableQED = ic.options().get<bool>("disable-qed"); //TODO: QED implementation to be tested`
`55`	`55`	`}`
`@@ -59,7 +59,7 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer`
`59`	`59`	`if (mFinished) {`
`60`	`60`	`return;`
`61`	`61`	`}`
`62`		`- LOG(INFO) << "FV0DPLDigitizerTask:run";`
	`62`	`+ LOG(DEBUG) << "FV0DPLDigitizerTask:run";`
`63`	`63`
`64`	`64`	`// read collision context from input`
`65`	`65`	`auto context = pc.inputs().get<o2::steer::DigitizationContext*>("collisioncontext");`
`@@ -83,14 +83,14 @@ class FV0DPLDigitizerTask : public o2::base::BaseDPLDigitizer`
`83`	`83`	`for (auto& part : eventParts[collID]) {`
`84`	`84`	`hits.clear();`
`85`	`85`	`context->retrieveHits(mSimChains, "FV0Hit", part.sourceID, part.entryID, &hits);`
`86`		`- LOG(INFO) << "[FV0] For collision " << collID << " eventID " << part.entryID << " found " << hits.size() << " hits ";`
	`86`	`+ LOG(DEBUG) << "[FV0] For collision " << collID << " eventID " << part.entryID << " found " << hits.size() << " hits ";`
`87`	`87`
`88`	`88`	`// call actual digitization procedure`
`89`	`89`	`mDigitizer.setEventId(part.entryID);`
`90`	`90`	`mDigitizer.setSrcId(part.sourceID);`
`91`	`91`	`mDigitizer.process(hits, mDigitsBC, mDigitsCh, mDigitsTrig, mLabels);`
`92`	`92`	`}`
`93`		`- LOG(INFO) << "[FV0] Has " << mDigitsBC.size() << " BC elements, " << mDigitsCh.size() << " mDigitsCh elements";`
	`93`	`+ LOG(DEBUG) << "[FV0] Has " << mDigitsBC.size() << " BC elements, " << mDigitsCh.size() << " mDigitsCh elements";`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`o2::InteractionTimeRecord terminateIR;`