[QC-1041] Reset checkrunner (#2062)

* add a reset method to Check and CheckInterface.

* explanations

* format

Author: Barthelemy

Framework/include/QualityControl/Check.h

@@ -60,6 +60,7 @@ class Check
    * Expected to run in the init phase of the FairDevice
    */
   void init();
+  void reset();
 
   core::QualityObjectsType check(std::map<std::string, std::shared_ptr<o2::quality_control::core::MonitorObject>>& moMap);
 

Framework/include/QualityControl/CheckInterface.h

@@ -60,14 +60,20 @@ class CheckInterface : public UserCodeInterface
   ///                    parameter is to be used to pass the result of the check of the same class.
   virtual void beautify(std::shared_ptr<core::MonitorObject> mo, core::Quality checkResult) = 0;
 
+  /// \brief Reset the state of this Check.
+  ///
+  /// This method should reset the state, if any, of the Check implemented here.
+  /// It will typically be called in between runs.
+  /// For example, if you have counters or you keep the state of an object from one call to the other,
+  /// then this should be reset here.
+  virtual void reset(); // not fully abstract because we don't want to change all the existing subclasses
+
   /// \brief Returns the name of the class that can be treated by this check.
   ///
   /// The name of the class returned by this method will be checked against the MonitorObject's encapsulated
   /// object's class. If it is the same or a parent then the check will be applied. Therefore, this method
   /// must return the highest class in the hierarchy that this check can use.
   /// If the class does not override it, we return "TObject".
-  ///
-  /// \author Barthelemy von Haller
   virtual std::string getAcceptedType();
 
   void setActivity(std::shared_ptr<core::Activity> activity) { mActivity = activity; }

Framework/src/Check.cxx

@@ -85,6 +85,11 @@ void Check::init()
   ILOG(Info, Devel) << ENDM;
 }
 
+void Check::reset()
+{
+  mCheckInterface->reset();
+}
+
 QualityObjectsType Check::check(std::map<std::string, std::shared_ptr<MonitorObject>>& moMap)
 {
   if (mCheckInterface == nullptr) {

Framework/src/CheckInterface.cxx

@@ -45,4 +45,9 @@ void CheckInterface::configure()
   // noop, override it if you want.
 }
 
+void CheckInterface::reset()
+{
+  // noop, override it if you want.
+}
+
 } // namespace o2::quality_control::checker

Framework/src/CheckRunner.cxx

@@ -533,11 +533,12 @@ void CheckRunner::stop()
 
 void CheckRunner::reset()
 {
-  ILOG(Info, Devel) << "Reset" << ENDM;
-
   try {
     mCollector.reset();
     mActivity = make_shared<Activity>();
+    for (auto& [checkName, check] : mChecks) {
+      check.reset();
+    }
   } catch (...) {
     // we catch here because we don't know where it will go in DPL's CallbackService
     ILOG(Error, Support) << "Error caught in reset() : "

Modules/Skeleton/include/Skeleton/SkeletonCheck.h

@@ -37,6 +37,7 @@ class SkeletonCheck : public o2::quality_control::checker::CheckInterface
   Quality check(std::map<std::string, std::shared_ptr<MonitorObject>>* moMap) override;
   void beautify(std::shared_ptr<MonitorObject> mo, Quality checkResult = Quality::Null) override;
   std::string getAcceptedType() override;
+  void reset() override;
 
   ClassDefOverride(SkeletonCheck, 2);
 };

Modules/Skeleton/src/SkeletonCheck.cxx

@@ -88,4 +88,9 @@ void SkeletonCheck::beautify(std::shared_ptr<MonitorObject> mo, Quality checkRes
   }
 }
 
+void SkeletonCheck::reset()
+{
+  ILOG(Debug, Devel) << "SkeletonCheck::reset" << ENDM;
+}
+
 } // namespace o2::quality_control_modules::skeleton