HLTTauDQM: conditional plotting

cms-sw · Jun 15, 2020 · 75cdd2f · 75cdd2f
1 parent df8e771
commit 75cdd2f
Show file tree

Hide file tree

Showing 14 changed files with 349 additions and 311 deletions.
diff --git a/DQMOffline/Trigger/interface/HLTTauDQML1Plotter.h b/DQMOffline/Trigger/interface/HLTTauDQML1Plotter.h
@@ -3,6 +3,7 @@
 #define HLTTauDQML1Plotter_h
 
 #include "DQMOffline/Trigger/interface/HLTTauDQMPlotter.h"
+#include "DQMOffline/Trigger/interface/IWrapper.h"
 
 #include "FWCore/Utilities/interface/InputTag.h"
 
@@ -25,7 +26,7 @@ class HLTTauDQML1Plotter : private HLTTauDQMPlotter {
 
   using HLTTauDQMPlotter::isValid;
 
-  void bookHistograms(DQMStore::IBooker& iBooker);
+  void bookHistograms(IWrapper &iWrapper, DQMStore::IBooker &iBooker);
   void analyze(const edm::Event& iEvent, const edm::EventSetup& iSetup, const HLTTauDQMOfflineObjects& refC);
 
 private:

diff --git a/DQMOffline/Trigger/interface/HLTTauDQMOfflineSource.h b/DQMOffline/Trigger/interface/HLTTauDQMOfflineSource.h
@@ -13,6 +13,7 @@
 #include "DQMServices/Core/interface/DQMEDAnalyzer.h"
 
 //Plotters
+#include "DQMOffline/Trigger/interface/IWrapper.h"
 #include "DQMOffline/Trigger/interface/HLTTauDQML1Plotter.h"
 #include "DQMOffline/Trigger/interface/HLTTauDQMPathPlotter.h"
 #include "DQMOffline/Trigger/interface/HLTTauDQMPathSummaryPlotter.h"
@@ -44,12 +45,13 @@ class HLTTauDQMOfflineSource : public DQMEDAnalyzer {
   edm::EDGetTokenT<trigger::TriggerEvent> triggerEventToken_;
 
   // For path plotters
-  const std::string pathRegexString_;
   const std::regex pathRegex_;
   const int nPtBins_, nEtaBins_, nPhiBins_;
   const double ptMax_, highPtMax_, l1MatchDr_, hltMatchDr_;
   const std::string dqmBaseFolder_;
 
+  IWrapper *iWrapper;
+
   HLTConfigProvider HLTCP_;
 
   //Reference

diff --git a/DQMOffline/Trigger/interface/HLTTauDQMPathPlotter.h b/DQMOffline/Trigger/interface/HLTTauDQMPathPlotter.h
@@ -4,6 +4,7 @@
 
 #include "DQMOffline/Trigger/interface/HLTTauDQMPlotter.h"
 #include "DQMOffline/Trigger/interface/HLTTauDQMPath.h"
+#include "DQMOffline/Trigger/interface/IWrapper.h"
 
 namespace edm {
   class Event;
@@ -35,7 +36,7 @@ class HLTTauDQMPathPlotter : private HLTTauDQMPlotter {
 
   using HLTTauDQMPlotter::isValid;
 
-  void bookHistograms(DQMStore::IBooker &iBooker);
+  void bookHistograms(IWrapper & iWrapper, DQMStore::IBooker &iBooker);
 
   void analyze(const edm::TriggerResults &triggerResults,
                const trigger::TriggerEvent &triggerEvent,

diff --git a/DQMOffline/Trigger/interface/HLTTauDQMPathSummaryPlotter.h b/DQMOffline/Trigger/interface/HLTTauDQMPathSummaryPlotter.h
@@ -3,6 +3,7 @@
 #define HLTTauDQMPathSummaryPlotter_h
 
 #include "DQMOffline/Trigger/interface/HLTTauDQMPlotter.h"
+#include "DQMOffline/Trigger/interface/IWrapper.h"
 
 #include <vector>
 
@@ -25,7 +26,7 @@ class HLTTauDQMPathSummaryPlotter : private HLTTauDQMPlotter {
   using HLTTauDQMPlotter::isValid;
 
   void setPathObjects(const std::vector<const HLTTauDQMPath*>& pathObjects) { pathObjects_ = pathObjects; }
-  void bookHistograms(DQMStore::IBooker& iBooker);
+  void bookHistograms(IWrapper &iWrapper, DQMStore::IBooker &iBooker);
 
   void analyze(const edm::TriggerResults& triggerResults,
                const trigger::TriggerEvent& triggerEvent,

diff --git a/DQMOffline/Trigger/interface/HLTTauDQMTagAndProbePlotter.h b/DQMOffline/Trigger/interface/HLTTauDQMTagAndProbePlotter.h
@@ -4,6 +4,7 @@
 
 #include "DQMOffline/Trigger/interface/HLTTauDQMPlotter.h"
 #include "DQMOffline/Trigger/interface/HLTTauDQMPath.h"
+#include "DQMOffline/Trigger/interface/IWrapper.h"
 
 //#include "CommonTools/TriggerUtils/interface/GenericTriggerEventFlag.h"
 #include "FWCore/Common/interface/TriggerNames.h"
@@ -29,7 +30,7 @@ class HLTTauDQMTagAndProbePlotter : private HLTTauDQMPlotter {
 
   using HLTTauDQMPlotter::isValid;
 
-  void bookHistograms(DQMStore::IBooker &iBooker, edm::Run const &iRun, edm::EventSetup const &iSetup);
+  void bookHistograms(IWrapper &iWrapper, DQMStore::IBooker &iBooker, edm::Run const &iRun, edm::EventSetup const &iSetup);
 
   void analyze(edm::Event const &iEvent,
                const edm::TriggerResults &triggerResults,

diff --git a/DQMOffline/Trigger/interface/IWrapper.h b/DQMOffline/Trigger/interface/IWrapper.h
@@ -0,0 +1,26 @@
+#ifndef __IWrapper__
+#define __IWrapper__
+
+#include "FWCore/ParameterSet/interface/ParameterSet.h"
+
+#include "DQMServices/Core/interface/DQMStore.h"
+typedef dqm::legacy::DQMStore DQMStore;
+#include "DQMServices/Core/interface/MonitorElement.h"
+typedef dqm::legacy::MonitorElement MonitorElement;
+
+enum PL {kEverything, kVital};
+
+class IWrapper {
+ public:
+  IWrapper(const edm::ParameterSet&);
+  ~IWrapper();
+
+  MonitorElement* book1D(DQMStore::IBooker& iBooker, TString const &name, TString const &title, int const nchX, double const lowX, double const highX, int level = kEverything);
+  MonitorElement* book2D(DQMStore::IBooker& iBooker, TString const &name, TString const &title, int nchX, double lowX, double highX, int nchY, double lowY, double highY, int level = kEverything);
+
+ private:
+  PL plotlevel;
+  int cAllHistograms;
+  int cPlottedHistograms;
+};
+#endif
diff --git a/DQMOffline/Trigger/plugins/HLTTauDQMOfflineSource.cc b/DQMOffline/Trigger/plugins/HLTTauDQMOfflineSource.cc
@@ -18,8 +18,7 @@ HLTTauDQMOfflineSource::HLTTauDQMOfflineSource(const edm::ParameterSet& ps)
       triggerResultsToken_(consumes<edm::TriggerResults>(triggerResultsSrc_)),
       triggerEventSrc_(ps.getUntrackedParameter<edm::InputTag>("TriggerEventSrc")),
       triggerEventToken_(consumes<trigger::TriggerEvent>(triggerEventSrc_)),
-      pathRegexString_(ps.getUntrackedParameter<std::string>("Paths")),
-      pathRegex_(pathRegexString_),
+      pathRegex_(ps.getUntrackedParameter<std::string>("Paths")),
       nPtBins_(ps.getUntrackedParameter<int>("PtHistoBins", 20)),
       nEtaBins_(ps.getUntrackedParameter<int>("EtaHistoBins", 12)),
       nPhiBins_(ps.getUntrackedParameter<int>("PhiHistoBins", 18)),
@@ -33,6 +32,8 @@ HLTTauDQMOfflineSource::HLTTauDQMOfflineSource(const edm::ParameterSet& ps)
   edm::ParameterSet matching = ps.getParameter<edm::ParameterSet>("Matching");
   doRefAnalysis_ = matching.getUntrackedParameter<bool>("doMatching");
 
+  iWrapper = new IWrapper(ps);
+
   if (ps.exists("L1Plotter") && !ps.exists("TagAndProbe")) {
     l1Plotter_ = std::make_unique<HLTTauDQML1Plotter>(ps.getUntrackedParameter<edm::ParameterSet>("L1Plotter"),
                                                       consumesCollector(),
@@ -76,7 +77,7 @@ void HLTTauDQMOfflineSource::dqmBeginRun(const edm::Run& iRun, const edm::EventS
         // Find all paths to monitor
         std::vector<std::string> foundPaths;
         std::smatch what;
-        LogDebug("HLTTauDQMOffline") << "Looking for paths with regex " << pathRegexString_;
+        LogDebug("HLTTauDQMOffline") << "Looking for paths with regex " << pathRegex_;
         for (const std::string& pathName : HLTCP_.triggerNames()) {
           if (std::regex_search(pathName, what, pathRegex_)) {
             LogDebug("HLTTauDQMOffline") << "Found path " << pathName;
@@ -164,16 +165,16 @@ void HLTTauDQMOfflineSource::bookHistograms(DQMStore::IBooker& iBooker,
                                             const edm::Run& iRun,
                                             const EventSetup& iSetup) {
   if (l1Plotter_) {
-    l1Plotter_->bookHistograms(iBooker);
+    l1Plotter_->bookHistograms(*iWrapper,iBooker);
   }
   for (auto& pathPlotter : pathPlotters_) {
-    pathPlotter.bookHistograms(iBooker);
+    pathPlotter.bookHistograms(*iWrapper,iBooker);
   }
   for (auto& tpPlotter : tagandprobePlotters_) {
-    tpPlotter->bookHistograms(iBooker, iRun, iSetup);
+    tpPlotter->bookHistograms(*iWrapper,iBooker, iRun, iSetup);
   }
   if (pathSummaryPlotter_) {
-    pathSummaryPlotter_->bookHistograms(iBooker);
+    pathSummaryPlotter_->bookHistograms(*iWrapper,iBooker);
   }
 }
 

diff --git a/DQMOffline/Trigger/plugins/HLTTauRefProducer.cc b/DQMOffline/Trigger/plugins/HLTTauRefProducer.cc
@@ -185,6 +185,7 @@ void HLTTauRefProducer::doPFTaus(edm::StreamID iID, edm::Event& iEvent) const {
           pftau.phi() > phiMinPFTau_ && pftau.phi() < phiMaxPFTau_) {
         reco::PFTauRef thePFTau{pftaus, i};
         bool passAll{true};
+
         for (auto const& token : PFTauDis_) {
           edm::Handle<reco::PFTauDiscriminator> pftaudis;
           if (iEvent.getByToken(token, pftaudis)) {
@@ -197,6 +198,7 @@ void HLTTauRefProducer::doPFTaus(edm::StreamID iID, edm::Event& iEvent) const {
             break;
           }
         }
+
         int idx = 0;
         for (auto const& token : PFTauDisCont_) {
           edm::Handle<reco::TauDiscriminatorContainer> pftaudis;

diff --git a/DQMOffline/Trigger/python/HLTTauDQMOffline_cfi.py b/DQMOffline/Trigger/python/HLTTauDQMOffline_cfi.py
@@ -1,24 +1,18 @@
 import FWCore.ParameterSet.Config as cms
-from Configuration.ProcessModifiers.tau_readOldDiscriminatorFormat_cff import tau_readOldDiscriminatorFormat
 
 hltTauDQMofflineProcess = "HLT"
 
 #Ref Objects-------------------------------------------------------------------------------------------------------
 TauRefProducer = cms.EDProducer("HLTTauRefProducer",
 
                     PFTaus = cms.untracked.PSet(
+                            PFTauDiscriminatorContainers  = cms.untracked.VInputTag(),
+                            PFTauDiscriminatorContainerWPs  = cms.untracked.vstring(),
                             PFTauDiscriminators = cms.untracked.VInputTag(
-                                    cms.InputTag("hpsPFTauDiscriminationByDecayModeFinding")
-                            ),
-                            PFTauDiscriminatorContainers = cms.untracked.VInputTag(
-                                    cms.InputTag("hpsPFTauBasicDiscriminators"),
-                                    cms.InputTag("hpsPFTauDiscriminationByMuonRejection3"),
-                                    cms.InputTag("hpsPFTauDiscriminationByMVA6ElectronRejection")
-                            ),
-                            PFTauDiscriminatorContainerWPs =  cms.untracked.vstring(
-                                    "ByLooseCombinedIsolationDBSumPtCorr3Hits",
-                                    "ByLooseMuonRejection3",
-                                    "_Tight"
+                                    cms.InputTag("hpsPFTauDiscriminationByDecayModeFinding"),
+                                    cms.InputTag("hpsPFTauDiscriminationByLooseCombinedIsolationDBSumPtCorr3Hits"),
+                                    cms.InputTag("hpsPFTauDiscriminationByLooseMuonRejection3"),
+                                    cms.InputTag("hpsPFTauDiscriminationByMVA6TightElectronRejection")
                             ),
                             doPFTaus = cms.untracked.bool(True),
                             ptMin = cms.untracked.double(15.0),
@@ -79,24 +73,15 @@
                     PhiMax = cms.untracked.double(3.15)
                   )
 
-tau_readOldDiscriminatorFormat.toModify(TauRefProducer.PFTaus,
-              PFTauDiscriminators = cms.untracked.VInputTag(
-                                    cms.InputTag("hpsPFTauDiscriminationByDecayModeFinding"),
-                                    cms.InputTag("hpsPFTauDiscriminationByLooseCombinedIsolationDBSumPtCorr3Hits"),
-                                    cms.InputTag("hpsPFTauDiscriminationByLooseMuonRejection3"),
-                                    cms.InputTag("hpsPFTauDiscriminationByMVA6TightElectronRejection"),
-                                    cms.InputTag("hpsPFTauDiscriminationByDecayModeFinding")
-                            ),
-              PFTauDiscriminatorContainers = cms.untracked.VInputTag(),
-              PFTauDiscriminatorContainerWPs = cms.untracked.vstring()
-)
-
 #----------------------------------MONITORS--------------------------------------------------------------------------
+kEverything = 0
+kVital      = 1
 
 from DQMServices.Core.DQMEDAnalyzer import DQMEDAnalyzer
 hltTauOfflineMonitor_PFTaus = DQMEDAnalyzer('HLTTauDQMOfflineSource',
     HLTProcessName = cms.untracked.string(hltTauDQMofflineProcess),
     DQMBaseFolder = cms.untracked.string("HLT/TAU/PFTaus"),
+    PlotLevel = cms.untracked.int32(kVital),
     TriggerResultsSrc = cms.untracked.InputTag("TriggerResults", "", hltTauDQMofflineProcess),
     TriggerEventSrc = cms.untracked.InputTag("hltTriggerSummaryAOD", "", hltTauDQMofflineProcess),
     L1Plotter = cms.untracked.PSet(