removing older checks TrackletPerTriggerCheck & TrackletsPerTimeFrame…

…Check

Modules/TRD/CMakeLists.txt

@@ -2,7 +2,7 @@
 
 add_library(O2QcTRD)
 
-target_sources(O2QcTRD PRIVATE src/TrackletCountCheck.cxx  src/TrackletsPerTimeFrameCheck.cxx  src/TrackletPerTriggerCheck.cxx  src/TRDReductor.cxx src/PulsePositionCheck.cxx  src/TrackingTask.cxx  src/PulseHeightTrackMatch.cxx  src/TrackletsCheck.cxx  src/TrackletsTask.cxx  src/PulseHeightCheck.cxx  src/PulseHeight.cxx  src/RawData.cxx  src/DigitsTask.cxx
+target_sources(O2QcTRD PRIVATE src/TrackletCountCheck.cxx  src/TRDReductor.cxx src/PulsePositionCheck.cxx  src/TrackingTask.cxx  src/PulseHeightTrackMatch.cxx  src/TrackletsCheck.cxx  src/TrackletsTask.cxx  src/PulseHeightCheck.cxx  src/PulseHeight.cxx  src/RawData.cxx  src/DigitsTask.cxx
                              src/DigitsCheck.cxx src/TRDTrending.cxx src/TrendingTaskConfigTRD.cxx src/PulseHeightPostProcessing.cxx)
 
 target_include_directories(
@@ -21,8 +21,6 @@ install(TARGETS O2QcTRD
 add_root_dictionary(O2QcTRD
                     HEADERS
   include/TRD/TrackletCountCheck.h
-  include/TRD/TrackletsPerTimeFrameCheck.h
-  include/TRD/TrackletPerTriggerCheck.h
   include/TRD/TRDReductor.h
   include/TRD/PulsePositionCheck.h
   include/TRD/TrackingTask.h

Modules/TRD/include/TRD/LinkDef.h

@@ -17,7 +17,5 @@
 #pragma link C++ class o2::quality_control_modules::trd::TrackingTask + ;
 #pragma link C++ class o2::quality_control_modules::trd::PulsePositionCheck + ;
 #pragma link C++ class o2::quality_control_modules::trd::TRDReductor + ;
-#pragma link C++ class o2::quality_control_modules::trd::TrackletPerTriggerCheck + ;
-#pragma link C++ class o2::quality_control_modules::trd::TrackletsPerTimeFrameCheck + ;
 #pragma link C++ class o2::quality_control_modules::trd::TrackletCountCheck + ;
 #endif

Modules/TRD/include/TRD/TrackletCountCheck.h

@@ -11,7 +11,7 @@
 
 ///
 /// \file   TrackletCountCheck.h
-/// \author My Name
+/// \author Deependra Sharma @IITB
 ///
 
 #ifndef QC_MODULE_TRD_TRDTRACKLETCOUNTCHECK_H
@@ -23,7 +23,7 @@ namespace o2::quality_control_modules::trd
 {
 
 /// \brief  Example QC Check
-/// \author My Name
+/// \author Deependra Sharma @IITB
 class TrackletCountCheck : public o2::quality_control::checker::CheckInterface
 {
  public:

Modules/TRD/src/TrackletCountCheck.cxx

@@ -11,7 +11,7 @@
 
 ///
 /// \file   TrackletCountCheck.cxx
-/// \author My Name
+/// \author Deependra Sharma @IITB
 ///
 
 #include "TRD/TrackletCountCheck.h"
@@ -58,7 +58,7 @@ Quality TrackletCountCheck::check(std::map<std::string, std::shared_ptr<MonitorO
   mResultPerTimeFrame = Quality::Null;
 
   for (auto& [moName, mo] : *moMap) {
-
+    ILOG(Debug, Support) << "moName =" << moName << ENDM;
     (void)moName;
     if (mo->getName() == "trackletsperevent") {
       auto* h = dynamic_cast<TH1F*>(mo->getObject());
@@ -85,18 +85,18 @@ Quality TrackletCountCheck::check(std::map<std::string, std::shared_ptr<MonitorO
       // Warning about triggers without any tracklets
       int UnderFlowTrackletsPerTrigger = h->GetBinContent(0);
       if (UnderFlowTrackletsPerTrigger > 0.) {
-        msg1->AddText(TString::Format("Triggers without Tracklets: %d", UnderFlowTrackletsPerTrigger));
+        msg1->AddText(TString::Format("Number of Triggers without Tracklets: %d", UnderFlowTrackletsPerTrigger));
       }
 
       // applying check
       float MeanTrackletPertrigger = h->GetMean();
       if (MeanTrackletPertrigger > mThresholdMeanLowPerTrigger && MeanTrackletPertrigger < mThresholdMeanHighPerTrigger) {
-        TText* Checkmsg = msg1->AddText("Mean Per Trigger is found in bound region: ok");
+        TText* Checkmsg = msg1->AddText(TString::Format("Mean Per Trigger: %f is found in bound region [%f, %f]", MeanTrackletPertrigger, mThresholdMeanLowPerTrigger, mThresholdMeanHighPerTrigger));
         Checkmsg->SetTextColor(kGreen);
         mResultPertrigger = Quality::Good;
       } else {
         mResultPertrigger = Quality::Bad;
-        TText* Checkmsg = msg1->AddText("Mean PerTrigger is not found in bound region: not ok");
+        TText* Checkmsg = msg1->AddText(TString::Format("Mean Per Trigger: %f is not found in bound region [%f, %f]", MeanTrackletPertrigger, mThresholdMeanLowPerTrigger, mThresholdMeanHighPerTrigger));
         Checkmsg->SetTextColor(kRed);
         mResultPertrigger.addReason(FlagReasonFactory::Unknown(), "MeanTrackletPertrigger is not in bound region");
       }
@@ -115,18 +115,18 @@ Quality TrackletCountCheck::check(std::map<std::string, std::shared_ptr<MonitorO
       // Warning about TimeFrame without any tracklets
       int UnderFlowTrackletsPertrigger = h2->GetBinContent(0);
       if (UnderFlowTrackletsPertrigger > 0.) {
-        msg2->AddText(TString::Format("TimeFrame without Tracklets: %d", UnderFlowTrackletsPertrigger));
+        msg2->AddText(TString::Format("Number of TimeFrames without Tracklets: %d", UnderFlowTrackletsPertrigger));
       }
 
       // applying check
       float MeanTrackletPerTimeFrame = h2->GetMean();
       if (MeanTrackletPerTimeFrame > mThresholdMeanLowPerTimeFrame && MeanTrackletPerTimeFrame < mThresholdMeanHighPerTimeFrame) {
-        TText* Checkmsg2 = msg2->AddText("Mean Per Timeframe is found in bound region: ok");
+        TText* Checkmsg2 = msg2->AddText(TString::Format("Mean Per Timeframe: %f is found in bound region [%f, %f]", MeanTrackletPerTimeFrame, mThresholdMeanLowPerTimeFrame, mThresholdMeanHighPerTimeFrame));
         Checkmsg2->SetTextColor(kGreen);
         mResultPerTimeFrame = Quality::Good;
       } else {
         mResultPerTimeFrame = Quality::Bad;
-        TText* Checkmsg2 = msg2->AddText("Mean per Timeframe is not found in bound region: not ok");
+        TText* Checkmsg2 = msg2->AddText(TString::Format("Mean per Timeframe: %f is not found in bound region[%f, %f]", MeanTrackletPerTimeFrame, mThresholdMeanLowPerTimeFrame, mThresholdMeanHighPerTimeFrame));
         Checkmsg2->SetTextColor(kRed);
         mResultPerTimeFrame.addReason(FlagReasonFactory::Unknown(), "MeanTrackletPerTimeFrame is not in bound region");
       }