From c41f052d42fd1d5aab5fb4f1ab8058c113be867b Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Sun, 17 Oct 2021 21:34:04 -0500 Subject: [PATCH 1/8] Start writing configuration for intime fcls --- .../standard/g4/g4_simphotontime_filter.fcl | 68 ++++++++++++++++--- .../prodcorsika_proton_intime_filter.fcl | 66 +++++++++++++++--- 2 files changed, 117 insertions(+), 17 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl index 32d91286b..6977c7bb1 100644 --- a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl @@ -1,16 +1,68 @@ #include "standard_g4_sbnd.fcl" -physics.producers.larg4outtime: @local::physics.producers.largeant -physics.producers.larg4outtime.InputLabels: [ "GenInTimeSorter:outtime" ] +# Add a geant4 process, that will run only on the outtime cosmics +physics.producers.loader: { module_type: "PhysListLoader" } +physics.producers.larg4outtime: @local::sbnd_larg4 +physics.producers.ionandscintouttime: @local::sbnd_ionandscint +physics.producers.pdfastsimouttime: @local::sbnd_pdfastsim_par +physics.producers.simdriftouttime: @local::sbnd_simdrift +# Set the appropriate input labels, to run geant4 only on the outtime cosmics +physics.producers.larg4outtime.inputCollections: [ "GenInTimeSorter:outtime" ] +physics.producers.ionandscintouttime.InputCollections: ["larg4outtime"] +physics.producers.pdfastsimouttime.SimulationLabel: "ionandscintouttime:priorSCE" +physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime" + +# Add a process that merges the intime and outtime geant4 collections physics.producers.largeant: { module_type: "MergeSimSources" - InputSourcesLabels: [ "larg4intime","larg4outtime"] - TrackIDOffsets: [ 10000000,20000000 ] + InputSourcesLabels: [ "larg4intime", "larg4outtime"] + TrackIDOffsets: [ 10000000, 20000000 ] } -physics.simulate: [ rns, larg4outtime, largeant, mcreco ] +# Tell MCReco +# physics.producers.mcreco.MCParticleLabel: "largeant" +# physics.producers.mcreco.SimChannelLabel: "largeant" + +# Add all these new modules to the simulate path +physics.simulate: [ rns + , loader + , larg4outtime + , ionandscintouttime + , pdfastsimouttime + , simdriftouttime + , largeant + # , mcreco + , rns ] + +# Drop the intime and outtime collections, which have now been +# been merged into a 'largeant' collection +# outputs.out1.outputCommands: [ "keep *_*_*_*", +# "drop *_larg4intime_*_*", +# "drop *_larg4outtime_*_*"] + +outputs.out1.outputCommands: [ "keep *_*_*_*"] + +# Remove unnecesary processes +# physics.producers.ionandscint: @erase +# physics.producers.ionandscintout: @erase +# physics.producers.pdfastsim: @erase +# physics.producers.pdfastsimout: @erase +# physics.producers.simdrift: @erase + + + +# physics.producers.larg4outtime: @local::physics.producers.largeant +# physics.producers.larg4outtime.InputLabels: [ "GenInTimeSorter:outtime" ] + +# physics.producers.largeant: { +# module_type: "MergeSimSources" +# InputSourcesLabels: [ "larg4intime","larg4outtime"] +# TrackIDOffsets: [ 10000000,20000000 ] +# } + +# physics.simulate: [ rns, larg4outtime, largeant, mcreco ] -outputs.out1.outputCommands: [ "keep *_*_*_*", - "drop *_larg4intime_*_*", - "drop *_larg4outtime_*_*"] +# outputs.out1.outputCommands: [ "keep *_*_*_*", +# "drop *_larg4intime_*_*", +# "drop *_larg4outtime_*_*"] diff --git a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl index 610334ba8..1dcb0b941 100644 --- a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl @@ -1,6 +1,10 @@ #include "filtersgenintime_sbnd.fcl" #include "filterssimphotonlitetime_sbnd.fcl" -#include "largeantmodules_sbnd.fcl" + +#include "larg4_sbnd.fcl" +#include "ionandscint_sbnd.fcl" +#include "PDFastSim_sbnd.fcl" +#include "simdrift_sbnd.fcl" #include "prodcorsika_cosmics_proton.fcl" @@ -12,17 +16,61 @@ services: { @table::sbnd_g4_services } -physics.producers.larg4intime: @local::sbnd_largeant +# Rename generator to corsika +physics.producers.corsika: @local::physics.producers.generator + +# Add a generation filter, that splits in intime and outtime cosmics physics.filters.GenInTimeSorter: @local::sbnd_filtergenintime + +# Add a geant4 process, only on the intime cosmics +physics.producers.loader: { module_type: "PhysListLoader" } +physics.producers.larg4intime: @local::sbnd_larg4 +physics.producers.ionandscintintime: @local::sbnd_ionandscint +physics.producers.pdfastsimintime: @local::sbnd_pdfastsim_par +physics.producers.simdriftintime: @local::sbnd_simdrift + +# Add a filter on the geant4 intime output, based on sim photons physics.filters.timefilter: @local::sbnd_timefilterssimphotonlitetime -physics.producers.corsika: @local::physics.producers.generator +# Add all these new modules to the simulate path +physics.simulate: [ corsika + , GenInTimeSorter + , loader + , larg4intime + , ionandscintintime + , pdfastsimintime + , simdriftintime + , timefilter + , rns ] + +# Remove unnecesary processes +physics.producers.generator: @erase + +# Set the appropriate input labels +physics.producers.larg4intime.inputCollections: ["GenInTimeSorter:intime"] +physics.producers.ionandscintintime.InputCollections: ["larg4intime"] +physics.producers.pdfastsimintime.SimulationLabel: "ionandscintintime:priorSCE" +physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" +physics.filters.timefilter.SimPhotonsLiteCollectionLabel: "pdfastsimintime" + +process_name: CosmicsCorsikaProtonGenAndG4InTime + + + + + + + +# physics.producers.larg4intime: @local::sbnd_largeant +# physics.filters.GenInTimeSorter: @local::sbnd_filtergenintime +# physics.filters.timefilter: @local::sbnd_timefilterssimphotonlitetime + -physics.simulate: [ corsika, GenInTimeSorter, larg4intime, timefilter, rns ] -outputs.out1.SelectEvents: [ "simulate" ] +# physics.simulate: [ corsika, GenInTimeSorter, larg4intime, timefilter, rns ] +# outputs.out1.SelectEvents: [ "simulate" ] -physics.producers.larg4intime.KeepParticlesInVolumes: ["volCryostat", "volTaggerTopHigh", "volTaggerTopLow", "volTaggerSideLeft", "volTaggerSideRight", "volTaggerFaceFront", "volTaggerFaceBack", "volTaggerBot"] -physics.producers.larg4intime.InputLabels: [ "GenInTimeSorter:intime" ] -#physics.producers.larg4intime.SparsifyTrajectory: "true" +# physics.producers.larg4intime.KeepParticlesInVolumes: ["volCryostat", "volTaggerTopHigh", "volTaggerTopLow", "volTaggerSideLeft", "volTaggerSideRight", "volTaggerFaceFront", "volTaggerFaceBack", "volTaggerBot"] +# physics.producers.larg4intime.InputLabels: [ "GenInTimeSorter:intime" ] +# #physics.producers.larg4intime.SparsifyTrajectory: "true" -process_name: CosmicsCorsikaCMCGenAndG4InTime +# process_name: CosmicsCorsikaCMCGenAndG4InTime From 8e8029da16a80f66b6839efba69600a70399c836 Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Tue, 19 Oct 2021 16:04:29 -0500 Subject: [PATCH 2/8] Add mergesimsources_sbnd.fcl with a basic config for the MergeSimSources module --- sbndcode/LArG4/mergesimsources_sbnd.fcl | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) create mode 100644 sbndcode/LArG4/mergesimsources_sbnd.fcl diff --git a/sbndcode/LArG4/mergesimsources_sbnd.fcl b/sbndcode/LArG4/mergesimsources_sbnd.fcl new file mode 100644 index 000000000..f9a425bf6 --- /dev/null +++ b/sbndcode/LArG4/mergesimsources_sbnd.fcl @@ -0,0 +1,17 @@ + +BEGIN_PROLOG + +sbnd_merge_sim_sources : { + module_type: "MergeSimSources" + FillMCParticles: false + FillSimPhotons: false + FillSimChannels: false + FillAuxDetSimChannels: false + FillSimEnergyDeposits: false + InputSourcesLabels: [ "larg4intime", "larg4outtime"] + TrackIDOffsets: [ 10000000, 20000000 ] + StoreReflected: true + EnergyDepositInstanceLabels: [ "priorSCE" ] +} + +END_PROLOG From b9afe7ac514ed005e7ed395e53a2ccde65bde02c Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Tue, 19 Oct 2021 16:05:05 -0500 Subject: [PATCH 3/8] First version of the prodcorsika fcl file for intime cosmics generation with the new larg4 --- .../prodcorsika_proton_intime_filter.fcl | 40 +++++++------------ 1 file changed, 15 insertions(+), 25 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl index 1dcb0b941..ec932cca3 100644 --- a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl @@ -1,3 +1,11 @@ +# File: prodcorsika_proton_intime_filter.fcl +# Purpose: Generated CORSIKA events filtering out events with no cosmics intime +# +# This runs the new, refactored, LArG4 simulation. +# +# Ported from uBooNE gen-in-time fhicl for use by SBND +# by Gray Putnam + #include "filtersgenintime_sbnd.fcl" #include "filterssimphotonlitetime_sbnd.fcl" @@ -8,9 +16,8 @@ #include "prodcorsika_cosmics_proton.fcl" -# Ported from uBooNE gen-in-time fhicl for use by SBND -# by Gray Putnam +# Add the g4 services services: { @table::services @table::sbnd_g4_services @@ -27,7 +34,6 @@ physics.producers.loader: { module_type: "PhysListLoader" } physics.producers.larg4intime: @local::sbnd_larg4 physics.producers.ionandscintintime: @local::sbnd_ionandscint physics.producers.pdfastsimintime: @local::sbnd_pdfastsim_par -physics.producers.simdriftintime: @local::sbnd_simdrift # Add a filter on the geant4 intime output, based on sim photons physics.filters.timefilter: @local::sbnd_timefilterssimphotonlitetime @@ -39,38 +45,22 @@ physics.simulate: [ corsika , larg4intime , ionandscintintime , pdfastsimintime - , simdriftintime , timefilter - , rns ] + , rns + ] # Remove unnecesary processes physics.producers.generator: @erase # Set the appropriate input labels physics.producers.larg4intime.inputCollections: ["GenInTimeSorter:intime"] -physics.producers.ionandscintintime.InputCollections: ["larg4intime"] +physics.producers.ionandscintintime.InputModuleLabels: ["larg4intime"] physics.producers.pdfastsimintime.SimulationLabel: "ionandscintintime:priorSCE" -physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" physics.filters.timefilter.SimPhotonsLiteCollectionLabel: "pdfastsimintime" -process_name: CosmicsCorsikaProtonGenAndG4InTime - - +services.ParticleListAction.keepGenTrajectories: ["GenInTimeSorter"] +outputs.out1.SelectEvents: [ "simulate" ] +process_name: CosmicsCorsikaProtonGenAndG4InTime - - -# physics.producers.larg4intime: @local::sbnd_largeant -# physics.filters.GenInTimeSorter: @local::sbnd_filtergenintime -# physics.filters.timefilter: @local::sbnd_timefilterssimphotonlitetime - - -# physics.simulate: [ corsika, GenInTimeSorter, larg4intime, timefilter, rns ] -# outputs.out1.SelectEvents: [ "simulate" ] - -# physics.producers.larg4intime.KeepParticlesInVolumes: ["volCryostat", "volTaggerTopHigh", "volTaggerTopLow", "volTaggerSideLeft", "volTaggerSideRight", "volTaggerFaceFront", "volTaggerFaceBack", "volTaggerBot"] -# physics.producers.larg4intime.InputLabels: [ "GenInTimeSorter:intime" ] -# #physics.producers.larg4intime.SparsifyTrajectory: "true" - -# process_name: CosmicsCorsikaCMCGenAndG4InTime From 274086e6ed3bb80cdd457fb19f029f4f6366b38f Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Tue, 19 Oct 2021 17:20:10 -0500 Subject: [PATCH 4/8] Add g4 configuration for intime samples --- .../standard/g4/g4_simphotontime_filter.fcl | 130 ++++++++++++------ 1 file changed, 91 insertions(+), 39 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl index 6977c7bb1..a20d4e225 100644 --- a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl @@ -1,5 +1,15 @@ +# File: g4_simphotontime_filter.fcl +# Purpose: ? +# +# This runs the new, refactored, LArG4 simulation. + +#include "mergesimsources_sbnd.fcl" + #include "standard_g4_sbnd.fcl" +# Add process to finish the electron drift simulation for the intimes +physics.producers.simdriftintime: @local::sbnd_simdrift + # Add a geant4 process, that will run only on the outtime cosmics physics.producers.loader: { module_type: "PhysListLoader" } physics.producers.larg4outtime: @local::sbnd_larg4 @@ -7,62 +17,104 @@ physics.producers.ionandscintouttime: @local::sbnd_ionandscint physics.producers.pdfastsimouttime: @local::sbnd_pdfastsim_par physics.producers.simdriftouttime: @local::sbnd_simdrift +# Add processes for light simulation outside the active volume (AV) for the intimes +physics.producers.ionandscintoutintime: @local::sbnd_ionandscint_out +physics.producers.pdfastsimoutintime: @local::sbnd_pdfastsim_pvs +physics.producers.ionandscintoutintime.InputModuleLabels: ["larg4intime"] +physics.producers.pdfastsimoutintime.SimulationLabel: "ionandscintoutintime" + +# Add processes for light simulation outside the active volume (AV) for the outtimes +physics.producers.ionandscintoutouttime: @local::sbnd_ionandscint_out +physics.producers.pdfastsimoutouttime: @local::sbnd_pdfastsim_pvs +physics.producers.ionandscintoutouttime.InputModuleLabels: ["larg4outtime"] +physics.producers.pdfastsimoutouttime.SimulationLabel: "ionandscintoutouttime" + # Set the appropriate input labels, to run geant4 only on the outtime cosmics +physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" physics.producers.larg4outtime.inputCollections: [ "GenInTimeSorter:outtime" ] -physics.producers.ionandscintouttime.InputCollections: ["larg4outtime"] +physics.producers.ionandscintouttime.InputModuleLabels: ["larg4outtime"] physics.producers.pdfastsimouttime.SimulationLabel: "ionandscintouttime:priorSCE" physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime" -# Add a process that merges the intime and outtime geant4 collections -physics.producers.largeant: { - module_type: "MergeSimSources" - InputSourcesLabels: [ "larg4intime", "larg4outtime"] - TrackIDOffsets: [ 10000000, 20000000 ] -} +# Add a process that merges the MCParticles +physics.producers.largeant: @local::sbnd_merge_sim_sources +physics.producers.largeant.FillMCParticles: true +physics.producers.largeant.InputSourcesLabels: [ "larg4intime", "larg4outtime"] + +# Add a process that merges the SimEnergyDeposits +physics.producers.ionandscint: @local::sbnd_merge_sim_sources +physics.producers.ionandscint.FillSimEnergyDeposits: true +physics.producers.ionandscint.InputSourcesLabels: [ "ionandscintintime", "ionandscintouttime"] + +# Add a process that merges the SimChannels +physics.producers.simdrift: @local::sbnd_merge_sim_sources +physics.producers.simdrift.FillSimChannels: true +physics.producers.simdrift.InputSourcesLabels: [ "simdriftintime", "simdriftouttime"] + +# Add a process that merges the AuxDetSimChannels +physics.producers.crtsimch: @local::sbnd_merge_sim_sources +physics.producers.crtsimch.FillAuxDetSimChannels: true +physics.producers.crtsimch.InputSourcesLabels: [ "?", "?"] + +# Add a process that merges the SimPhotons inside the AV +physics.producers.pdfastsim: @local::sbnd_merge_sim_sources +physics.producers.pdfastsim.FillSimPhotons: true +physics.producers.pdfastsim.InputSourcesLabels: [ "pdfastsimintime", "pdfastsimouttime"] -# Tell MCReco -# physics.producers.mcreco.MCParticleLabel: "largeant" -# physics.producers.mcreco.SimChannelLabel: "largeant" +# Add a process that merges the SimPhotons outside the AV +physics.producers.pdfastsimout: @local::sbnd_merge_sim_sources +physics.producers.pdfastsimout.FillSimPhotons: true +physics.producers.pdfastsimout.InputSourcesLabels: [ "pdfastsimoutintime", "pdfastsimoutouttime"] # Add all these new modules to the simulate path physics.simulate: [ rns + ### Complete intime drift simulation + , simdriftintime + ### Do full Geant4 simulation for the outtimes , loader , larg4outtime , ionandscintouttime , pdfastsimouttime , simdriftouttime + ### Simulate the light outside the AV + , ionandscintoutintime + , pdfastsimoutintime + , ionandscintoutouttime + , pdfastsimoutouttime + ### Merge the intime and outtime paths , largeant - # , mcreco - , rns ] + , ionandscint + , simdrift + , pdfastsim + , pdfastsimout + ### Do truth-level reconstruction + , mcreco + ] + +services.ParticleListAction.keepGenTrajectories: ["GenInTimeSorter"] # Drop the intime and outtime collections, which have now been # been merged into a 'largeant' collection -# outputs.out1.outputCommands: [ "keep *_*_*_*", -# "drop *_larg4intime_*_*", -# "drop *_larg4outtime_*_*"] - -outputs.out1.outputCommands: [ "keep *_*_*_*"] +outputs.out1.outputCommands: [ "keep *_*_*_*" + # Drop G4 + , "drop *_larg4intime_*_*" + , "drop *_larg4outtime_*_*" + # Drop IonAndScint Inside AV + , "drop *_ionandscintintime_*_*" + , "drop *_ionandscintouttime_*_*" + # Drop PDFastSim Inside AV + , "drop *_pdfastsimintime_*_*" + , "drop *_pdfastsimouttime_*_*" + # Drop SimDrift Inside AV + , "drop *_simdriftintime_*_*" + , "drop *_simdriftouttime_*_*" + # Drop IonAndScint Outside AV + , "drop *_ionandscintoutintime_*_*" + , "drop *_ionandscintoutouttime_*_*" + # Drop PDFastSim Ouside AV + , "drop *_pdfastsimoutintime_*_*" + , "drop *_pdfastsimoutouttime_*_*" + ] # Remove unnecesary processes -# physics.producers.ionandscint: @erase -# physics.producers.ionandscintout: @erase -# physics.producers.pdfastsim: @erase -# physics.producers.pdfastsimout: @erase -# physics.producers.simdrift: @erase - - - -# physics.producers.larg4outtime: @local::physics.producers.largeant -# physics.producers.larg4outtime.InputLabels: [ "GenInTimeSorter:outtime" ] - -# physics.producers.largeant: { -# module_type: "MergeSimSources" -# InputSourcesLabels: [ "larg4intime","larg4outtime"] -# TrackIDOffsets: [ 10000000,20000000 ] -# } - -# physics.simulate: [ rns, larg4outtime, largeant, mcreco ] - -# outputs.out1.outputCommands: [ "keep *_*_*_*", -# "drop *_larg4intime_*_*", -# "drop *_larg4outtime_*_*"] +physics.producers.ionandscintout: @erase From 92cd5cffdfec7231f214d24fb98ea41de1c2a44d Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Tue, 19 Oct 2021 17:36:10 -0500 Subject: [PATCH 5/8] Clean up --- .../JobConfigurations/standard/g4/g4_simphotontime_filter.fcl | 3 ++- .../standard/gen/corsika/prodcorsika_proton_intime_filter.fcl | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl index a20d4e225..7668909e2 100644 --- a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl @@ -1,5 +1,6 @@ # File: g4_simphotontime_filter.fcl -# Purpose: ? +# Purpose: A geant4 fcl, supposed to run after prodcorsika_proton_intime_filter.fcl, or similar, +# for making cosmics in time samples. # # This runs the new, refactored, LArG4 simulation. diff --git a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl index ec932cca3..f965d2569 100644 --- a/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/gen/corsika/prodcorsika_proton_intime_filter.fcl @@ -1,5 +1,5 @@ # File: prodcorsika_proton_intime_filter.fcl -# Purpose: Generated CORSIKA events filtering out events with no cosmics intime +# Purpose: Generates CORSIKA events filtering out events with no cosmics intime # # This runs the new, refactored, LArG4 simulation. # From 6025ccdd38c5cc1fc59b6fdf0653ec4128710665 Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Sat, 23 Oct 2021 00:12:30 -0500 Subject: [PATCH 6/8] Clean up --- .../standard/g4/g4_simphotontime_filter.fcl | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl index 7668909e2..ce11cc638 100644 --- a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl @@ -18,6 +18,13 @@ physics.producers.ionandscintouttime: @local::sbnd_ionandscint physics.producers.pdfastsimouttime: @local::sbnd_pdfastsim_par physics.producers.simdriftouttime: @local::sbnd_simdrift +# Set the appropriate input labels, to run geant4 only on the outtime cosmics +physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" +physics.producers.larg4outtime.inputCollections: [ "GenInTimeSorter:outtime" ] +physics.producers.ionandscintouttime.InputModuleLabels: ["larg4outtime"] +physics.producers.pdfastsimouttime.SimulationLabel: "ionandscintouttime:priorSCE" +physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime" + # Add processes for light simulation outside the active volume (AV) for the intimes physics.producers.ionandscintoutintime: @local::sbnd_ionandscint_out physics.producers.pdfastsimoutintime: @local::sbnd_pdfastsim_pvs @@ -30,13 +37,6 @@ physics.producers.pdfastsimoutouttime: @local::sbnd_pdfastsim_pvs physics.producers.ionandscintoutouttime.InputModuleLabels: ["larg4outtime"] physics.producers.pdfastsimoutouttime.SimulationLabel: "ionandscintoutouttime" -# Set the appropriate input labels, to run geant4 only on the outtime cosmics -physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" -physics.producers.larg4outtime.inputCollections: [ "GenInTimeSorter:outtime" ] -physics.producers.ionandscintouttime.InputModuleLabels: ["larg4outtime"] -physics.producers.pdfastsimouttime.SimulationLabel: "ionandscintouttime:priorSCE" -physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime" - # Add a process that merges the MCParticles physics.producers.largeant: @local::sbnd_merge_sim_sources physics.producers.largeant.FillMCParticles: true @@ -52,7 +52,7 @@ physics.producers.simdrift: @local::sbnd_merge_sim_sources physics.producers.simdrift.FillSimChannels: true physics.producers.simdrift.InputSourcesLabels: [ "simdriftintime", "simdriftouttime"] -# Add a process that merges the AuxDetSimChannels +# Add a process that merges the AuxDetSimChannels TODO physics.producers.crtsimch: @local::sbnd_merge_sim_sources physics.producers.crtsimch.FillAuxDetSimChannels: true physics.producers.crtsimch.InputSourcesLabels: [ "?", "?"] From e2aa114b9894b3f9ee5fd4b6c784bccb8ce23a3e Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Sat, 23 Oct 2021 00:14:20 -0500 Subject: [PATCH 7/8] SimDrift needs to use priorSCE collection, as in PR #195 --- .../JobConfigurations/standard/g4/g4_simphotontime_filter.fcl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl index ce11cc638..8dbef235f 100644 --- a/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl +++ b/sbndcode/JobConfigurations/standard/g4/g4_simphotontime_filter.fcl @@ -19,11 +19,11 @@ physics.producers.pdfastsimouttime: @local::sbnd_pdfastsim_par physics.producers.simdriftouttime: @local::sbnd_simdrift # Set the appropriate input labels, to run geant4 only on the outtime cosmics -physics.producers.simdriftintime.SimulationLabel: "ionandscintintime" +physics.producers.simdriftintime.SimulationLabel: "ionandscintintime:priorSCE" physics.producers.larg4outtime.inputCollections: [ "GenInTimeSorter:outtime" ] physics.producers.ionandscintouttime.InputModuleLabels: ["larg4outtime"] physics.producers.pdfastsimouttime.SimulationLabel: "ionandscintouttime:priorSCE" -physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime" +physics.producers.simdriftouttime.SimulationLabel: "ionandscintouttime:priorSCE" # Add processes for light simulation outside the active volume (AV) for the intimes physics.producers.ionandscintoutintime: @local::sbnd_ionandscint_out From 9f057a41be0d1d955f04e60a9efd7f100aa6bae0 Mon Sep 17 00:00:00 2001 From: Marco Del Tutto Date: Mon, 1 Nov 2021 16:12:55 -0500 Subject: [PATCH 8/8] TFileService name to start with hists_* --- sbndcode/JobConfigurations/base/prodcosmics_corsika_sbnd.fcl | 2 +- .../JobConfigurations/base/prodgenie_bnb_nu_cosmic_sbnd.fcl | 2 +- .../JobConfigurations/base/prodgenie_common_cosmic_sbnd.fcl | 2 +- sbndcode/JobConfigurations/base/prodsingle_common_sbnd.fcl | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sbndcode/JobConfigurations/base/prodcosmics_corsika_sbnd.fcl b/sbndcode/JobConfigurations/base/prodcosmics_corsika_sbnd.fcl index b8c602b20..3fbaf1408 100644 --- a/sbndcode/JobConfigurations/base/prodcosmics_corsika_sbnd.fcl +++ b/sbndcode/JobConfigurations/base/prodcosmics_corsika_sbnd.fcl @@ -48,7 +48,7 @@ process_name: CosmicsGen services: { - TFileService: { fileName: "prodcosmics_corsika_sbnd_%p-%tc_hists.root" } + TFileService: { fileName: "hists_prodcosmics_corsika_sbnd_%p-%tc.root" } @table::sbnd_basic_services # from simulationservices_sbnd.fcl @table::sbnd_random_services # from simulationservices_sbnd.fcl @table::sbnd_services # from simulationservices_sbnd.fcl diff --git a/sbndcode/JobConfigurations/base/prodgenie_bnb_nu_cosmic_sbnd.fcl b/sbndcode/JobConfigurations/base/prodgenie_bnb_nu_cosmic_sbnd.fcl index 94056b5c3..7932470e4 100644 --- a/sbndcode/JobConfigurations/base/prodgenie_bnb_nu_cosmic_sbnd.fcl +++ b/sbndcode/JobConfigurations/base/prodgenie_bnb_nu_cosmic_sbnd.fcl @@ -25,7 +25,7 @@ process_name: GenieGen # output files -services.TFileService.fileName: "prodgenie_bnb_nu_cosmic_sbnd_%p-%tc_hists.root" +services.TFileService.fileName: "hists_prodgenie_bnb_nu_cosmic_sbnd_%p-%tc.root" outputs.out1.fileName: "prodgenie_bnb_nu_cosmic_sbnd_%p-%tc.root" physics.producers.generator: @local::sbnd_genie_simple diff --git a/sbndcode/JobConfigurations/base/prodgenie_common_cosmic_sbnd.fcl b/sbndcode/JobConfigurations/base/prodgenie_common_cosmic_sbnd.fcl index 096771545..f53d36747 100644 --- a/sbndcode/JobConfigurations/base/prodgenie_common_cosmic_sbnd.fcl +++ b/sbndcode/JobConfigurations/base/prodgenie_common_cosmic_sbnd.fcl @@ -46,7 +46,7 @@ process_name: GenieGen # services: { - TFileService: { fileName: "prodgenie_common_cosmic_sbnd_%p-%tc_hists.root" } + TFileService: { fileName: "hists_prodgenie_common_cosmic_sbnd_%p-%tc.root" } IFDH: {} # required by GENIEGen @table::sbnd_basic_services # from simulationservices_sbnd.fcl @table::sbnd_random_services # from simulationservices_sbnd.fcl diff --git a/sbndcode/JobConfigurations/base/prodsingle_common_sbnd.fcl b/sbndcode/JobConfigurations/base/prodsingle_common_sbnd.fcl index 66e82aec2..3310f6ca2 100644 --- a/sbndcode/JobConfigurations/base/prodsingle_common_sbnd.fcl +++ b/sbndcode/JobConfigurations/base/prodsingle_common_sbnd.fcl @@ -31,7 +31,7 @@ services: { @table::sbnd_basic_services @table::sbnd_random_services - TFileService: { fileName: "prodeminus_0.1_0.9_sbnd_%p-%tc_hists.root" } + TFileService: { fileName: "hists_prodeminus_0.1_0.9_sbnd_%p-%tc.root" } FileCatalogMetadata: @local::sbnd_file_catalog_mc }