From 86d19b1c19f9bff9bf6a64d9a69abb8ae3908fff Mon Sep 17 00:00:00 2001
From: mmusich <marco.musich@cern.ch>
Date: Wed, 6 Apr 2022 11:40:53 +0200
Subject: [PATCH] use new fill method and class templates

---
 .../SiStripApvGain_PayloadInspector.cc        | 10 +--
 .../SiStripLorentzAngle_PayloadInspector.cc   | 64 +++++++++----------
 2 files changed, 37 insertions(+), 37 deletions(-)

diff --git a/CondCore/SiStripPlugins/plugins/SiStripApvGain_PayloadInspector.cc b/CondCore/SiStripPlugins/plugins/SiStripApvGain_PayloadInspector.cc
index e58957630d3d4..1b61fd4f869ee 100644
--- a/CondCore/SiStripPlugins/plugins/SiStripApvGain_PayloadInspector.cc
+++ b/CondCore/SiStripPlugins/plugins/SiStripApvGain_PayloadInspector.cc
@@ -89,12 +89,13 @@ namespace {
     }  // fill
   };
 
-  class SiStripApvGainByPartition : public PlotImage<SiStripApvGain> {
+  class SiStripApvGainByPartition : public PlotImage<SiStripApvGain, SINGLE_IOV> {
   public:
-    SiStripApvGainByPartition() : PlotImage<SiStripApvGain>("SiStrip ApvGains By Partition") { setSingleIov(true); }
+    SiStripApvGainByPartition() : PlotImage<SiStripApvGain, SINGLE_IOV>("SiStrip ApvGains By Partition") {}
 
-    bool fill(const std::vector<std::tuple<cond::Time_t, cond::Hash>>& iovs) override {
-      auto iov = iovs.front();
+    bool fill() override {
+      auto tag = PlotBase::getTag<0>();
+      auto iov = tag.iovs.front();
       std::shared_ptr<SiStripApvGain> payload = fetchPayload(std::get<1>(iov));
       if (payload.get()) {
         SiStripApvGainContainer* objContainer =
@@ -106,7 +107,6 @@ namespace {
 
         std::string fileName(m_imageFileName);
         canvas.SaveAs(fileName.c_str());
-
       }  // payload
       return true;
     }  // fill
diff --git a/CondCore/SiStripPlugins/plugins/SiStripLorentzAngle_PayloadInspector.cc b/CondCore/SiStripPlugins/plugins/SiStripLorentzAngle_PayloadInspector.cc
index e87a05a83ddc3..e6b527948095d 100644
--- a/CondCore/SiStripPlugins/plugins/SiStripLorentzAngle_PayloadInspector.cc
+++ b/CondCore/SiStripPlugins/plugins/SiStripLorentzAngle_PayloadInspector.cc
@@ -55,52 +55,52 @@ namespace {
   /************************************************
     testing the machinery
   ************************************************/
-  class SiStripLorentzAngleTest : public PlotImage<SiStripLorentzAngle> {
+  class SiStripLorentzAngleTest : public PlotImage<SiStripLorentzAngle, SINGLE_IOV> {
   public:
-    SiStripLorentzAngleTest() : PlotImage<SiStripLorentzAngle>("SiStrip LorentzAngle values") { setSingleIov(true); }
+    SiStripLorentzAngleTest() : PlotImage<SiStripLorentzAngle, SINGLE_IOV>("SiStrip LorentzAngle values") {}
 
-    bool fill(const std::vector<std::tuple<cond::Time_t, cond::Hash> > &iovs) override {
-      for (auto const &iov : iovs) {
-        std::shared_ptr<SiStripLorentzAngle> payload = fetchPayload(std::get<1>(iov));
-        if (payload.get()) {
-          SiStripLorentzAngleContainer *objContainer =
-              new SiStripLorentzAngleContainer(payload, std::get<0>(iov), std::get<1>(iov));
-          //objContainer->printAll();
+    bool fill() override {
+      auto tag = PlotBase::getTag<0>();
+      auto iov = tag.iovs.front();
 
-          TCanvas canvas("Partion summary", "partition summary", 1200, 1000);
-          objContainer->fillSummary(canvas);
+      std::shared_ptr<SiStripLorentzAngle> payload = fetchPayload(std::get<1>(iov));
+      if (payload.get()) {
+        SiStripLorentzAngleContainer *objContainer =
+            new SiStripLorentzAngleContainer(payload, std::get<0>(iov), std::get<1>(iov));
+        //objContainer->printAll();
 
-          std::string fileName(m_imageFileName);
-          canvas.SaveAs(fileName.c_str());
+        TCanvas canvas("Partion summary", "partition summary", 1200, 1000);
+        objContainer->fillSummary(canvas);
 
-        }  // payload
-      }    // iovs
+        std::string fileName(m_imageFileName);
+        canvas.SaveAs(fileName.c_str());
+
+      }  // payload
       return true;
     }  // fill
   };
 
-  class SiStripLorentzAngleByPartition : public PlotImage<SiStripLorentzAngle> {
+  class SiStripLorentzAngleByPartition : public PlotImage<SiStripLorentzAngle, SINGLE_IOV> {
   public:
-    SiStripLorentzAngleByPartition() : PlotImage<SiStripLorentzAngle>("SiStrip LorentzAngle By Partition") {
-      setSingleIov(true);
-    }
+    SiStripLorentzAngleByPartition()
+        : PlotImage<SiStripLorentzAngle, SINGLE_IOV>("SiStrip LorentzAngle By Partition") {}
 
-    bool fill(const std::vector<std::tuple<cond::Time_t, cond::Hash> > &iovs) override {
-      for (auto const &iov : iovs) {
-        std::shared_ptr<SiStripLorentzAngle> payload = fetchPayload(std::get<1>(iov));
-        if (payload.get()) {
-          SiStripLorentzAngleContainer *objContainer =
-              new SiStripLorentzAngleContainer(payload, std::get<0>(iov), std::get<1>(iov));
-          objContainer->printAll();
+    bool fill() override {
+      auto tag = PlotBase::getTag<0>();
+      auto iov = tag.iovs.front();
 
-          TCanvas canvas("Partition summary", "partition summary", 1400, 1000);
-          objContainer->fillByPartition(canvas, 100, 0., 0.05);
+      std::shared_ptr<SiStripLorentzAngle> payload = fetchPayload(std::get<1>(iov));
+      if (payload.get()) {
+        SiStripLorentzAngleContainer *objContainer =
+            new SiStripLorentzAngleContainer(payload, std::get<0>(iov), std::get<1>(iov));
+        objContainer->printAll();
 
-          std::string fileName(m_imageFileName);
-          canvas.SaveAs(fileName.c_str());
+        TCanvas canvas("Partition summary", "partition summary", 1400, 1000);
+        objContainer->fillByPartition(canvas, 100, 0., 0.05);
 
-        }  // payload
-      }    // iovs
+        std::string fileName(m_imageFileName);
+        canvas.SaveAs(fileName.c_str());
+      }  // payload
       return true;
     }  // fill
   };