From 1118d9de16bcebf620bfa6d928298caa35f0c81b Mon Sep 17 00:00:00 2001
From: Matteo Vicenzi <matteo.vicenzi7@gmail.com>
Date: Mon, 7 Oct 2024 20:07:44 -0500
Subject: [PATCH 1/3] add missing class dictionary definitions

---
 icaruscode/IcarusObj/classes.h       | 1 +
 icaruscode/IcarusObj/classes_def.xml | 6 ++++++
 2 files changed, 7 insertions(+)

diff --git a/icaruscode/IcarusObj/classes.h b/icaruscode/IcarusObj/classes.h
index 7e3b391ce..05e6001ae 100644
--- a/icaruscode/IcarusObj/classes.h
+++ b/icaruscode/IcarusObj/classes.h
@@ -6,6 +6,7 @@
 #include "icaruscode/IcarusObj/CRTTPCMatchingInfo.h"
 #include "icaruscode/IcarusObj/OpDetWaveformMeta.h"
 #include "icaruscode/IcarusObj/PMTWaveformTimeCorrection.h"
+#include "icaruscode/IcarusObj/PMTBeamSignal.h"
 #include "icaruscode/IcarusObj/Hit.h"
 //#include "icaruscode/IcarusObj/CRTPMTMatching.h"
 
diff --git a/icaruscode/IcarusObj/classes_def.xml b/icaruscode/IcarusObj/classes_def.xml
index 47d627181..0dda646c4 100644
--- a/icaruscode/IcarusObj/classes_def.xml
+++ b/icaruscode/IcarusObj/classes_def.xml
@@ -41,6 +41,12 @@
   <class name="std::vector<icarus::timing::PMTWaveformTimeCorrection>"/>
   <class name="art::Wrapper<std::vector<icarus::timing::PMTWaveformTimeCorrection> >"/>
 
+  <!-- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -->
+  <!-- icarus::timing::PMTBeamSignal -->
+  <class name="art::Wrapper<vector<icarus::timing::PMTBeamSignal> >"/>
+  <class name="icarus::timing::PMTBeamSignal"/>
+  <class name="vector<icarus::timing::PMTBeamSignal>"/>
+
   <!-- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -->
   <!-- LArSoft object associations that are missing from LArSoft             -->
   

From 7166c8be743adfb3f687480e7f31a2fff79671bd Mon Sep 17 00:00:00 2001
From: Matteo Vicenzi <matteo.vicenzi7@gmail.com>
Date: Mon, 7 Oct 2024 20:23:15 -0500
Subject: [PATCH 2/3] remove useless debug flag

---
 icaruscode/PMT/OpReco/ICARUSFlashAssAna_module.cc  | 5 -----
 icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl | 1 -
 2 files changed, 6 deletions(-)

diff --git a/icaruscode/PMT/OpReco/ICARUSFlashAssAna_module.cc b/icaruscode/PMT/OpReco/ICARUSFlashAssAna_module.cc
index 3943a5664..9c976c74e 100644
--- a/icaruscode/PMT/OpReco/ICARUSFlashAssAna_module.cc
+++ b/icaruscode/PMT/OpReco/ICARUSFlashAssAna_module.cc
@@ -107,10 +107,6 @@ class opana::ICARUSFlashAssAna : public art::EDAnalyzer
         Name("PEOpHitThreshold"),
         Comment("Threshold in PE for an OpHit to be considered in the information calculated for a flash")};
 
-    fhicl::Atom<bool> Debug{
-        Name("Debug"),
-        Comment("Be more verbose"),
-        false};
   };
 
   using Parameters = art::EDAnalyzer::Table<Config>;
@@ -271,7 +267,6 @@ opana::ICARUSFlashAssAna::ICARUSFlashAssAna(Parameters const &config)
       fFlashLabels(config().FlashLabels()),
       fRWMLabel(config().RWMLabel()),
       fPEOpHitThreshold(config().PEOpHitThreshold()),
-      fDebug(config().Debug()),
       fGeom(lar::providerFrom<geo::Geometry>())
 {
 }
diff --git a/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl b/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
index e64474a00..68d7dc840 100644
--- a/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
+++ b/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
@@ -112,7 +112,6 @@ ICARUSFlashAssAna: {
     FlashLabels: ["opflashCryoE", "opflashCryoW"]
     RWMLabel: "beamTiming:RWM"
     PEOpHitThreshold: 0
-    Debug: false
 }
 
 ICARUSBeamStructureAna: {

From ff6808e91f92e6c36172bb18c0453af1ac2416cc Mon Sep 17 00:00:00 2001
From: Matteo Vicenzi <matteo.vicenzi7@gmail.com>
Date: Mon, 7 Oct 2024 20:24:13 -0500
Subject: [PATCH 3/3] disable waveform tree for standard running

---
 icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl b/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
index 68d7dc840..9f51e9e74 100644
--- a/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
+++ b/icaruscode/PMT/OpReco/fcl/icarus_opana_modules.fcl
@@ -103,7 +103,7 @@ ICARUSParticleAna: {
 ICARUSFlashAssAna: {
     module_type: "ICARUSFlashAssAna"
     TriggerLabel: "daqTrigger"
-    DumpWaveformsInfo: true
+    DumpWaveformsInfo: false
     SaveRawWaveforms: false
     UseSharedBaseline: true
     OpDetWaveformLabels: ["daqPMT"]