Skip to content

Commit e16887b

Browse files
committed
Merge remote-tracking branch 'upstream/develop' into feature/tjyang_omevd
2 parents a0cee84 + 15adbb5 commit e16887b

31 files changed

Lines changed: 387 additions & 156 deletions

sbndcode/Commissioning/fcls/run_hitdumper.fcl

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,7 @@
1414
#
1515
#include "caldata_sbnd.fcl"
1616
#include "hitfindermodules_sbnd.fcl"
17-
#include "cluster_sbnd.fcl"
18-
#include "trackfindermodules_sbnd.fcl"
19-
#include "calorimetry_sbnd.fcl"
20-
#include "showerfindermodules.fcl"
2117
#include "databaseutil_sbnd.fcl"
22-
#include "vertexfindermodules.fcl"
2318
#include "hitdumpermodule.fcl"
2419
#include "muontrackmodule.fcl"
2520
#include "ophit_finder_sbnd.fcl"

sbndcode/Commissioning/fcls/run_muontrack.fcl

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,7 @@
1313
#
1414
#include "caldata_sbnd.fcl"
1515
#include "hitfindermodules_sbnd.fcl"
16-
#include "cluster_sbnd.fcl"
17-
#include "trackfindermodules_sbnd.fcl"
18-
#include "calorimetry_sbnd.fcl"
19-
#include "showerfindermodules.fcl"
2016
#include "databaseutil_sbnd.fcl"
21-
#include "vertexfindermodules.fcl"
2217

2318
#include "muontrackmodule.fcl"
2419
#include "hitdumpermodule.fcl"

sbndcode/Decoders/TPC/SBNDTPCDecoder_module.cc

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,6 @@ daq::SBNDTPCDecoder::Config::Config(fhicl::ParameterSet const & param) {
101101
void daq::SBNDTPCDecoder::produce(art::Event & event)
102102
{
103103
auto daq_handle = event.getHandle<artdaq::Fragments>(_tag);
104-
if ( !daq_handle.isValid() ) {
105-
throw cet::exception("SBNDTPCDecoder_module ") << " invalid fragment handle";
106-
}
107104

108105
RDPmkr rdpm(event);
109106
TSPmkr tspm(event);
@@ -114,10 +111,17 @@ void daq::SBNDTPCDecoder::produce(art::Event & event)
114111
std::unique_ptr<RDTsAssocs> rdtsassoc_collection(new RDTsAssocs);
115112
std::unique_ptr<std::vector<tpcAnalysis::TPCDecodeAna>> header_collection(new std::vector<tpcAnalysis::TPCDecodeAna>);
116113

117-
for (auto const &rawfrag: *daq_handle) {
118-
process_fragment(event, rawfrag, rawdigit_collection, header_collection, rdpm, tspm, rdts_collection, rdtsassoc_collection);
114+
if ( daq_handle.isValid() ) {
115+
for (auto const &rawfrag: *daq_handle) {
116+
process_fragment(event, rawfrag, rawdigit_collection, header_collection, rdpm, tspm, rdts_collection, rdtsassoc_collection);
117+
}
119118
}
119+
else
120+
{
121+
mf::LogWarning("SBNDTPCDecoder_module") << " Invalid fragment handle: Skipping TPC digit decoding";
122+
}
120123

124+
121125
event.put(std::move(rawdigit_collection));
122126
event.put(std::move(rdts_collection));
123127
event.put(std::move(rdtsassoc_collection));

sbndcode/JobConfigurations/README.md

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,26 @@ At the time of writing, the core workflow (for BNB + Dirt + Cosmics) is the foll
1010
- `reco1_sce_lite.fcl`
1111
- `reco2_sce.fcl`
1212

13+
The intime workflow is as follows:
14+
15+
- `prodcorsika_proton_intime_filter_sce.fcl`
16+
- `g4_sce_simphotontime_filter_lite.fcl`
17+
- `detsim_sce_lite.fcl`
18+
- `reco1_sce_lite.fcl`
19+
- `reco2_sce.fcl`
20+
21+
For single generator workflows (like intrinsic neutrino samples) the workflow is the following:
22+
23+
- `<your-gen>.fcl`
24+
- `g4_sce_lite.fcl`
25+
- `detsim_sce_lite.fcl`
26+
- `reco1_sce_lite.fcl`
27+
- `reco2_sce.fcl`
28+
1329
This may well change over the coming months, and this README should be updated to reflect this.
1430

1531
** ALERT **
1632

1733
Due to changes implemented in sbndcode PRs #408 and #409 the 1D simulation fcls will not work out of the box, they will need editing by experts!
1834

19-
Henry Lay & Lynn Tung - Feb 2024
35+
Henry Lay, Lynn Tung, Bear Carlson - Feb 2024
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
#include "evd_sbnd.fcl"
2+
services.RawDrawingOptions.SeeBadChannels: true
3+
services.RawDrawingOptions.MinimumSignal: 0
4+
services.RawDrawingOptions.PedestalOption: 1
5+
Services.RawDrawingOptions.TotalTicks: 3415
6+
services.ColorDrawingOptions.ColorOrGrayScale: 0

sbndcode/JobConfigurations/base/prodgenie_rockbox_sbnd.fcl

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333

3434
#include "genie_sbnd.fcl"
3535
#include "larg4_sbnd.fcl"
36+
#include "mlreco_sbnd.fcl"
3637

3738

3839
process_name: GenieGen
@@ -84,11 +85,14 @@ physics:
8485

8586
# The geant4 step
8687
largeant: @local::sbnd_larg4
88+
89+
# Merge dropped and nominal MCParticle into separate list
90+
simplemerge: @local::sbnd_simplemerge
8791
}
8892

8993
#define the producer and filter modules for this path, order matters,
90-
simulatetpc: [ rns, generator, loader, largeant, tpcfilter ]
91-
simulatedirt: [ rns, generator, loader, largeant, "!tpcfilter", dirtfilter ]
94+
simulatetpc: [ rns, generator, loader, largeant, simplemerge, tpcfilter ]
95+
simulatedirt: [ rns, generator, loader, largeant, simplemerge, "!tpcfilter", dirtfilter ]
9296

9397
#define the output stream, there could be more than one if using filters
9498
# By deault Keep everything in one file:

sbndcode/JobConfigurations/base/reco_drops.fcl

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,11 @@ BEGIN_PROLOG
1919

2020
reco_drops: [ @sequence::detsim_drops,
2121
"drop raw::RawDigits_*_*_*",
22-
"drop raw::OpDetWaveforms_*_*_*" ]
22+
"drop raw::OpDetWaveforms_*_*_*",
23+
"drop *_cluster3d_*_*", #drop all mlreco output
24+
"drop *_simplemerge_*_*", #drop all mlreco output
25+
"drop *_sedlite_*_*" #drop all mlreco output
26+
]
2327

2428
END_PROLOG
2529

sbndcode/JobConfigurations/base/reco_sbnd.fcl

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424

2525
#include "caldata_sbnd.fcl"
2626
#include "hitfindermodules_sbnd.fcl"
27-
#include "cluster_sbnd.fcl"
2827
#include "trackfindermodules_sbnd.fcl"
2928
#include "calorimetry_sbnd.fcl"
3029
#include "showerfindermodules_sbnd.fcl"
@@ -48,12 +47,14 @@
4847
#include "scecorrections.fcl"
4948
#include "flashmatch_sbnd.fcl"
5049

51-
###include "hitparticleassociations.fcl"
50+
#include "mlreco_sbnd.fcl"
51+
#include "cluster_sbnd.fcl"
5252

5353
#include "sbnd_trackcalo_skimmer.fcl"
5454
#include "crtskim_sbnd.fcl"
5555
#include "pmtskim_sbnd.fcl"
5656

57+
5758
process_name: Reco
5859

5960
services:
@@ -124,10 +125,10 @@ physics:
124125

125126
### cluster-finder producers
126127
#fuzzycluster: @local::sbnd_fuzzycluster
127-
linecluster: @local::sbnd_linecluster
128+
#linecluster: @local::sbnd_linecluster
128129
#cccluster: @local::sbnd_clustercrawler
129130
#spacepts: @local::sbnd_spacepts
130-
blurredcluster: @local::sbnd_blurredcluster
131+
#blurredcluster: @local::sbnd_blurredcluster
131132

132133
### pandora
133134
pandora: @local::sbnd_pandora
@@ -192,6 +193,9 @@ physics:
192193
### Uncalibrated calorimetry producer for calibration caloskimmer
193194
caloskimCalorimetry: @local::caloskim_calorimetry
194195

196+
#ML Reco
197+
cluster3d: @local::sbnd_cluster3d
198+
195199
}
196200

197201
#define the producer and filter modules for this path, order matters,
@@ -214,6 +218,7 @@ physics:
214218
# , corner
215219
# , fuzzycluster
216220
, crtstrips
221+
, cluster3d
217222
]
218223

219224
reco2_no_opt0finder: [
@@ -250,9 +255,11 @@ physics:
250255
caloskim: @local::caloskim_nodigits_goldentracks
251256
crtskim: @local::crtskim_sbnd
252257
pmtskim: @local::pmtskim_sbnd
258+
supera: @local::sbnd_supera
253259
}
254260

255-
caloskimana: [ caloskim, crtskim, pmtskim ]
261+
caloskimana: [ caloskim, crtskim, pmtskim ]
262+
superaana: [ supera ] #outputs larcv.root for use in mlreco
256263

257264
#define the output stream, there could be more than one if using filters
258265
stream1: [ out1 ]
@@ -280,8 +287,10 @@ outputs:
280287
"drop *raw::RawDigits*_*_*_*",
281288
#"drop sim::SimChannels_*_*_*",
282289
"drop sim::OpDetBacktrackerRecords_*_*_*",
283-
"drop *_linecluster_*_*",
284-
"drop *_fasthit_*_*"
290+
#"drop *_linecluster_*_*",
291+
"drop *_fasthit_*_*",
292+
"drop *_cluster3d_*_*", #drop all mlreco output
293+
"drop *_simplemerge_*_*" #drop all mlreco output
285294
]
286295

287296

@@ -316,7 +325,7 @@ physics.producers.fasthit.DigitModuleLabel: "simtpc2d:daq"
316325
#physics.producers.corner.CornerAlgParamSet.RawDataModuleLabel: "daq"
317326
#physics.producers.corner.CornerAlgParamSet.CalDataModuleLabel: "caldata"
318327
#
319-
physics.producers.linecluster.HitFinderModuleLabel: "gaushit"
328+
#physics.producers.linecluster.HitFinderModuleLabel: "gaushit"
320329

321330
#physics.producers.fuzzycluster.HitsModuleLabel: "gaushit"
322331
#physics.producers.fuzzycluster.CalDataModuleLabel: "caldata"

sbndcode/JobConfigurations/standard/g4/g4_dirt_filter.fcl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
#include "standard_g4_sbnd.fcl"
44

55
physics.simulate: [ rns
6+
, sedlite #SEDLite reduces info from SED
67
, ionandscint
78
, ionandscintout
89
, pdfastsim

sbndcode/JobConfigurations/standard/g4/g4_enterstpc_filter.fcl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
physics.simulate: [ rns
66
, loader
77
, largeant
8+
, simplemerge #merges dropped and nominal MCParticle
9+
, sedlite
810
, ionandscint
911
, ionandscintout
1012
, pdfastsim

0 commit comments

Comments
 (0)