-
Notifications
You must be signed in to change notification settings - Fork 43
Updates for first reprocessing of PDS products in spring productions #876
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
mvicenzi
wants to merge
69
commits into
release/SBN2025A
Choose a base branch
from
feature/mvicenzi_repro_step1
base: release/SBN2025A
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
3cab2e9
fix sim label for PDFastSim
mvicenzi b6c5c1a
detsim_opdetonly_icarus.fcl: simulate only optical detector digitization
PetrilloAtWork ae195e1
Changed determination of the end of single photoelectron response tem…
PetrilloAtWork b129249
Updated `FastGaussianNoiseGeneratorAlg` documentation
PetrilloAtWork 3f467fb
SelectCathodeCrossingGenParticles: selects generated events with part…
PetrilloAtWork 0dc5ec4
Merge branch 'mvicenzi/quick_fix' into feature/mvicenzi_repro_step1
mvicenzi 0d3e9ff
make MC2-var4 tune standard
mvicenzi 1428556
simulate pmt timing delays in mc
mvicenzi 9d43105
disable pmt mc noise for overlays
mvicenzi d655888
add ophituncorrected + ophit flow for mc
mvicenzi 6f0342b
update stage0 flow for overlays
mvicenzi 7e655c6
small fix
mvicenzi 8726590
prelimanary reco gain tuning
mvicenzi 0fe2b8e
add SPRRun3, make SPRRun2 default
mvicenzi a138d80
nullify flash calibration
mvicenzi e9a9bea
add run3 opdet tune, make run2 default
mvicenzi e918fe8
change run3->run4, update gain
mvicenzi 677563f
Merge branch 'release/SBN2025A' into feature/mvicenzi_repro_step1
mvicenzi 99f4413
add run4 detsim fhicls
mvicenzi b90cfb5
first attempt at detsim reprocessing fhicl
mvicenzi 468cb3f
fix typo
mvicenzi 9667657
Tell CMake of sbnalg dependency
PetrilloAtWork 8c1e40b
Trigger simulation modules: support input tag with process name
PetrilloAtWork 937b4b6
Apply suggestions from GIT Copilot code review
PetrilloAtWork ab9ecf2
Merge pull request #870 from PetrilloAtWork/feature/gp_LVDSgatesFix
mvicenzi a80e88b
fix to detsim reprocessing
mvicenzi fe5e072
reprocessing opdetonly overlay fhicl
mvicenzi 2142449
leave default tags
mvicenzi 0c07aeb
add initial recalibrator module
mvicenzi 16be497
fix compilation for recalibrator
mvicenzi 8dfa2db
link online corrections service to recalibrator
mvicenzi d96f212
link locally-provided old timing corrections
mvicenzi 76c5cd0
fix timing recalibration
mvicenzi 46942ec
remove custom config, fixed indentation
mvicenzi 8d29faa
chop off flat waveform tail in overlays
mvicenzi 23005d1
restore legacy defaults for backward compatibility
mvicenzi 2f8cf32
fixed documentation
mvicenzi d18c784
add opdetonly MCstage0 fhicls for reprocessing
mvicenzi 28f94e0
fix stage0 reprocessing fhicls
mvicenzi 86cb353
adjust stage1 compatibility
mvicenzi 09ea82e
add data reprocessing fhicl(s)
mvicenzi 5af5235
update file names
mvicenzi f574863
first commit, compiles and does nothing
57e45ba
changed namespaces
eb6bdb1
basic struct/map for holding spe areas
0bd1735
working version getting area from database and initial database file
6987683
updated tags in db file
667d576
add fcl file for debugging spe area service
d7b1c3a
ICARUSFlashFinder: added Doxygen-style header
PetrilloAtWork d91e636
ICARUSFlashFinder: turned into shared module
PetrilloAtWork 4a92ddf
ICARUSFlashFinder: FHiCL configuration validation
PetrilloAtWork e433639
ICARUSFlashFinder: code updates
PetrilloAtWork 93fcc2c
ICARUSFlashFinder: removed hit recalibration (PECalib)
PetrilloAtWork 4d8d9f4
dump_opflashes_icarus.fcl: dump reconstructed flashes on text file
PetrilloAtWork 15541a4
Adopting corrections from GitHub Copilot
PetrilloAtWork a019654
Removed obsolete icarus_flashcalib.fcl
PetrilloAtWork 6c188f2
PMT calibration service code review
PetrilloAtWork 4db5efb
update pmt calibration service provider
ba917d5
Merge branch 'release/SBN2025A' into feature/mvicenzi_repro_step1
mvicenzi 74b471e
Merge pull request #880 from carriganm95/carriganm95/speAreaTool
mvicenzi a4b8813
removed db file + copilot suggestions
mvicenzi 7efe1eb
interface SPEArea DB with ophit recalibrator
mvicenzi 5ad6287
fix cout of current areatag
mvicenzi c2df7b8
rework stage0 optical workflow for data, mc, overlay
mvicenzi d7c5c93
simulate pmt timing delays only for overlay flow
mvicenzi 2a7f75e
remove ophituncurrected from legacy mc flow
mvicenzi c435bf2
tune mcophit
mvicenzi d5024e9
make applytimingdelays explicit
mvicenzi 3139d6f
add run3/4 stage0 overlay light tunes
mvicenzi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| #include "detsim_2d_icarus_refactored.fcl" | ||
|
|
||
| # Run3/4 optical tune | ||
| physics.producers.opdaq: @local::icarus_simpmt_run4 | ||
| physics.producers.opdaq.ApplyTimingDelays: false # keep false for non-overlay MC |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,8 @@ | ||
| #include "detsim_2d_icarus_refactored.fcl" | ||
|
|
||
| physics.producers.daq: @local::icarus_simwire_wirecell_shifted_overlay | ||
|
|
||
| # turn off mc noise on pmt waveforms | ||
| physics.producers.opdaq: @local::icarus_simpmt_nonoise | ||
| physics.producers.opdaq.ApplyTimingDelays: true | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| #include "detsim_2d_icarus_refactored_overlay.fcl" | ||
|
|
||
| # Run3/4 optical tune (with no noise) | ||
| physics.producers.opdaq: @local::icarus_simpmt_run4_nonoise | ||
| physics.producers.opdaq.ApplyTimingDelays: true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| #include "detsim_2d_icarus_refactored_yzsim.fcl" | ||
|
|
||
| physics.producers.daq.wcls_main.params.YZScaleMapJson: "yzmap_gain_icarus_v4_run4.json" | ||
|
|
||
| # Run3/4 optical tune | ||
| physics.producers.opdaq: @local::icarus_simpmt_run4 | ||
| physics.producers.opdaq.ApplyTimingDelays: false # keep false for non-overlay MC |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 changes: 4 additions & 0 deletions
4
fcl/detsim/detsim_2d_icarus_refactored_yzsim_notrigger_overlay.fcl
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| #include "detsim_2d_icarus_refactored_yzsim_notrigger.fcl" | ||
|
|
||
| physics.producers.daq: @local::icarus_simwire_wirecell_yz_overlay | ||
|
|
||
| # turn off mc noise on pmt waveforms | ||
| physics.producers.opdaq: @local::icarus_simpmt_nonoise | ||
| physics.producers.opdaq.ApplyTimingDelays: true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| #include "detsim_2d_icarus_refactored_yzsim.fcl" | ||
|
|
||
| physics.producers.daq: @local::icarus_simwire_wirecell_yz_overlay | ||
|
|
||
| # turn off mc noise on pmt waveforms | ||
| physics.producers.opdaq: @local::icarus_simpmt_nonoise | ||
| physics.producers.opdaq.ApplyTimingDelays: true |
4 changes: 4 additions & 0 deletions
4
fcl/detsim/detsim_2d_icarus_refactored_yzsim_overlay_Run4.fcl
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| #include "detsim_2d_icarus_refactored_yzsim_overlay.fcl" | ||
|
|
||
| physics.producers.daq.wcls_main.params.YZScaleMapJson: "yzmap_gain_icarus_v4_run4.json" | ||
|
|
||
| # Run3/4 optical tune (with no noise) | ||
| physics.producers.opdaq: @local::icarus_simpmt_run4_nonoise | ||
| physics.producers.opdaq.ApplyTimingDelays: true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Install fcl files in /job subdirectory. | ||
| install_fhicl() | ||
|
|
||
| # Also put a copy in the source tree. | ||
|
|
||
| FILE(GLOB fcl_files *.fcl) | ||
| install_source( EXTRAS ${fcl_files} ) | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| # | ||
| # File: detsim_opdetonly_icarus.fcl | ||
| # Purpose: Simulation of the optical detector response. | ||
| # Date: January 27, 2025 | ||
| # Author: Gianluca Petrillo (petrillo@slac.stanford.edu) | ||
| # | ||
| # This job configuration runs the standard detector simulation, like | ||
| # `detsim_1d_icarus.fcl` does, but only for the optical detector. | ||
| # | ||
| # | ||
| # Output | ||
| # ------- | ||
| # | ||
| # * `std::vector<raw::OpDetWaveform>` (`opdaq`): raw PMT waveforms | ||
| # | ||
| # | ||
| # | ||
| # | ||
| # Changes | ||
| # -------- | ||
| # | ||
| # 20250127 (petrillo@slac.stanford.edu) [v1.0] | ||
| # original version (from `detsim_1d_icarus.fcl`) | ||
| # | ||
|
|
||
| #include "detsim_1d_icarus.fcl" | ||
|
|
||
| process_name: OpDetSim | ||
|
|
||
| services.DetPedestalService: @erase | ||
| services.ChannelStatusService: @erase | ||
| services.IICARUSChannelMap: @erase | ||
| services.SignalShapingICARUSService: @erase | ||
|
|
||
| # run only the `opdaq` module: | ||
| physics.producers: { | ||
| opdaq: @local::physics.producers.opdaq # simplify configuration removing all other producers | ||
| } | ||
| physics.simulate: [ opdaq ] | ||
|
|
||
140 changes: 140 additions & 0 deletions
140
fcl/detsim/partial/detsim_opdetonly_icarus_overlay_reprocessing_run2.fcl
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again some comment words about what this configuration does and what needs reprocessing would help. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,140 @@ | ||
| #include "detsim_2d_icarus_refactored.fcl" | ||
|
|
||
| process_name: OpDetSim | ||
|
|
||
| services.DetPedestalService: @erase | ||
| services.ChannelStatusService: @erase | ||
| services.SignalShapingICARUSService: @erase | ||
|
|
||
| source.inputCommands: [ | ||
| "keep *", | ||
|
|
||
| # `stage0` comes from decoding data: should be kept! | ||
| # `MCstage0` comes running stage0 on the overlay products: | ||
| # so pmt/trigger/crt stuff should be thrown out! | ||
| # note: I would really like to drop these things as well: | ||
| # "drop raw::OpDetWaveform*_opdaq_*_*", # drop old MC waveforms (if any) | ||
| # "drop *_pmtfixedthrinit_*_*", # drop triggersim product | ||
| # "drop *_pmtlvdsgatesinit_*_*", # drop triggersim product | ||
| # "drop *_pmttriggerwindowsinit_*_*", # drop triggersim product | ||
| # "drop *_triggersimgatesinit_*_*", # drop triggersim product | ||
| # "drop *_emuTriggerUnshifted_*_*", # drop triggersim product | ||
| # but I can't: it triggers art genocidal insticts thus removing all | ||
| # downstream products in the dependency tree that we still need. | ||
|
|
||
| "drop raw::OpDetWaveform*_shifted_*_*", # drop old MC (shifted) waveforms (if any) | ||
| "drop *_overlayOpWaveforms_*_*", # drop old Overlay waveforms (if any) | ||
| "drop *_ophit*_*_*", # drop MCstage0 ophits | ||
| "drop *_mcophit_*_*", # drop MCstageo mcophit | ||
| "drop *_opflash*_*_*", # drop MCsstage0 east/west opflashes | ||
|
|
||
| "drop *_pmtfixedthr_*_*", # drop MCstage0 trigger product | ||
| "drop *_pmtlvdsgates_*_*", # drop MCstage0 trigger product | ||
| "drop *_pmtlvdsgates_*_*", # drop MCstage0 trigger product | ||
| "drop *_pmtbaselines_*_MCstage0", # drop MCstage0 baselines, not stage0! | ||
| "drop *_pmttriggerwindows_*_*", # drop MCstage0 trigger product | ||
| "drop *_emuTrigger_*_*", # drop MCstage0 trigger product | ||
|
|
||
| "drop *_crtdaq_*_*", # drop CRT detsim product | ||
| "drop *_mccrthit_*_*", # drop CRT mc hits | ||
| "drop *_overlayCRTHit_*_*", # drop CRT overlay hits | ||
| "drop *_crttrack_*_*", # drop CRT tracks (MCstage0) | ||
| "drop *_crtpmt_*_*" # drop CRTPMT matches (MCstage0) | ||
| ] | ||
|
|
||
| # Running only the optical/trigger/crt modules | ||
| # Simplify configuration removing all other producers (tpc) | ||
|
|
||
| physics.producers: { | ||
|
|
||
| # opdaq Run-1/2 tune for overlays (no noise) | ||
| opdaq: @local::icarus_simpmt_run2_nonoise | ||
|
|
||
| # triggersim producers | ||
| @table::icarus_shifting_triggersim.producers | ||
|
|
||
| # crt hits | ||
| crtdaq: @local::icarus_crtsim | ||
|
|
||
| } | ||
|
|
||
| # since some "old" products need to be kept in the tree | ||
| # we have products with the same tag, and different process_name | ||
| # make sure to select exactly what you need! | ||
|
|
||
| # build waveforms from simPhotons (already shifted) | ||
| physics.producers.opdaq.InputModule: "shifted::DetSim" | ||
| physics.producers.opdaq.ApplyTimingDelays: true | ||
|
|
||
| # make sure the triggersim chain used the "new" products | ||
| # instead of the "old" ones which I cannot drop on input | ||
| physics.producers.pmtfixedthrinit.OpticalWaveforms: "opdaq::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.pmtlvdsgatesinit.TriggerGatesTag: "pmtfixedthrinit::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.pmttriggerwindowsinit.TriggerGatesTag: "pmtlvdsgatesinit::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.triggersimgatesinit.BeamGateTag: "beamgate" # keep using old Gen product | ||
| physics.producers.emuTriggerUnshifted.BeamGates: "triggersimgatesinit::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.emuTriggerUnshifted.TriggerGatesTag: "pmttriggerwindowsinit::OpDetSim" # this was re-generated with same tag | ||
|
|
||
| # need to update the shifting module to pick up new labels | ||
| # some were regenerated (tag still good), others were previously shifted | ||
| # applying a second shift is okay: everything is consistent | ||
| # (eg. waveforms made from `shifted` photons) | ||
| # NOTE: still shifing energy deposits although tpc info not regenerated | ||
| # however forced to use `filtersed` as `shifted` has been consumed | ||
|
|
||
| physics.producers.shifted.InputTriggerLabel: "emuTriggerUnshifted::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.shifted.InitAuxDetSimChannelLabel: "shifted::DetSim" # need to start from previously-shifted, not `genericcrt` | ||
| physics.producers.shifted.InitBeamGateInfoLabel: "triggersimgatesinit::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.shifted.InitSimEnergyDepositLabel: "filtersed::DetSim" # need to start from previously-shifted, but `shifted` no longer available | ||
| physics.producers.shifted.InitSimEnergyDepositLiteLabel: "filtersed::DetSim" # need to start from previously-shifted, but `shifted` no longer available | ||
| physics.producers.shifted.InitSimPhotonsLabel: "shifted::DetSim" # need to start from previously-shifted, not `pdfastsim` | ||
| physics.producers.shifted.InitWaveformLabel: "opdaq::OpDetSim" # this was re-generated with same tag | ||
|
|
||
| # need to update shiting of priorSCE deposits: | ||
| # this is important if future reprocessings will need to regenerate SimPhotons | ||
|
|
||
| physics.producers.shiftedpriorSCE.InputTriggerLabel: "emuTriggerUnshifted::OpDetSim" # this was re-generated with same tag | ||
| physics.producers.shiftedpriorSCE.InitSimEnergyDepositLabel: "shiftedpriorSCE::DetSim" # need to start from previously-shifted, not `ionization:priorSCE` | ||
|
|
||
| # and finally make sure the crtsim also uses the newly shifted products | ||
| physics.producers.crtsim.G4ModuleLabel: "shifted::OpDetSim" | ||
|
|
||
| # The reprocessing flow is the following: | ||
| # - build new pmt waveforms from photons | ||
| # - simulate the trigger | ||
| # - shift all products to the trigger: | ||
| # -- we are shifting photons, pmt waveforms, energy depositis, aux sim channels | ||
| # -- note: since tpc is not re-simulated, no longer fully coherent with trigger time | ||
| # - build CRT data from aux sim channels | ||
|
|
||
| physics.simulate: [ opdaq, @sequence::icarus_shifting_triggersim.path, crtdaq ] | ||
|
|
||
|
|
||
| # drop duplicate produtcs in output | ||
| # this doesn't trigger art genocidal instincts | ||
|
|
||
| outputs.rootoutput.outputCommands: [ | ||
| "keep *", | ||
|
|
||
| # drop `opdaq`, but keep `shifted` for overlaying | ||
| "drop *_opdaq_*_*", | ||
|
|
||
| # drop old trigger products now that new ones exist | ||
| # specify the old process name "DetSim" | ||
| "drop *_pmtfixedthrinit_*_DetSim", | ||
| "drop *_pmtlvdsgatesinit_*_DetSim", | ||
| "drop *_pmttriggerwindowsinit_*_DetSim", | ||
| "drop *_triggersimgatesinit_*_DetSim", | ||
| "drop *_emuTriggerUnshifted_*_DetSim", | ||
|
|
||
| # drop old "shifted" products since we shifted things again | ||
| "drop sim::SimEnergyDeposits_filtersed_*_DetSim", # this has been regenerated into `shifted` | ||
| # mcreco looks for `filtersed` in stage1: potentially can be changed to look for `shifted`? | ||
| # however since tpc reco products are not shifted, better to keep old deposits for that matching | ||
| # "drop sim::SimEnergyDepositLites_filtersed_*_DetSim", # this has been regenerated into `shifted` | ||
| "drop sim::SimEnergyDeposits_shiftedpriorSCE_*_DetSim", # this has been regenerated into the same tag | ||
| "drop sim::AuxDetSimChannel_shifted_*_DetSim", # this has been regenerated into the same tag | ||
| "drop sim::SimPhotons_shifted_*_DetSim", # this has been regenerated into the same tag | ||
| "drop sim::BeamGateInfo_shifted_*_DetSim" # this has been regenerated into the same tag | ||
|
|
||
| ] |
6 changes: 6 additions & 0 deletions
6
fcl/detsim/partial/detsim_opdetonly_icarus_overlay_reprocessing_run4.fcl
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| #include "detsim_opdetonly_icarus_overlay_reprocessing_run2.fcl" | ||
|
|
||
| # switch to Run-4 tune | ||
| physics.producers.opdaq: @local::icarus_simpmt_run4_nonoise | ||
| physics.producers.opdaq.InputModule: "shifted::DetSim" | ||
| physics.producers.opdaq.ApplyTimingDelays: true |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Install fcl files in /job subdirectory. | ||
| install_fhicl() | ||
|
|
||
| # Also put a copy in the source tree. | ||
|
|
||
| FILE(GLOB fcl_files *.fcl) | ||
| install_source( EXTRAS ${fcl_files} ) | ||
|
|
25 changes: 25 additions & 0 deletions
25
fcl/overlays/partial/overlay_opdetonly_waveforms_reprocessing.fcl
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| #include "overlay_waveforms.fcl" | ||
|
|
||
| process_name: OpOverlay | ||
|
|
||
| # only perform PMT and CRT overlays after their detsim reprocessing | ||
| # previous reprocessing step should have cleaned-up duplicates already | ||
| # so technically no need to update any tag names w.r.t the default file | ||
|
|
||
| #physics.producers.mccrthit.CrtModuleLabel: "crtdaq" # from previous reprocessing | ||
| #physics.producers.mccrthit.TriggerLabel: "daqTrigger" | ||
|
|
||
| #physics.producers.overlayOpWaveforms.PMTWaveDataLabel: "daqPMT" # from data decoding | ||
| #phsyics.producers.overlayOpWaveforms.PMTWaveSimLabel: "shifted" # from previous reprocessing | ||
| #phsyics.producers.overlayOpWaveforms.PMTWaveBaseLabel: "pmtbaselines" # from data decoding | ||
|
|
||
| #physics.producers.overlayCRTHit.CRTHitInputLabels: ["crthit", # from data decoding | ||
| # "mccrthit"] # from current process | ||
|
|
||
| # process includes: | ||
| # - make crt hits on mc | ||
| # - overlay pmt waveforms | ||
| # - overlay crt hits | ||
|
|
||
| physics.reco: [ mccrthit, overlayOpWaveforms, overlayCRTHit] | ||
|
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which of the dozen DetSim configurations is the "reference" MC (no overlay) one now? we still have to choose a run period, isn't it? I am more than a bit lost.
It may be that this needs to be turned into a Run2 configuration, using as base
detsim_2d_icarus_refactored.fcllikedetsim_opdetonly_icarus_overlay_reprocessing_run2.fcldoes (and call thisdetsim_opdetonly_icarus_run2.fcl).