Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds shower normalization capabilities using TPC, YZ, and drift SQLite normalization tools to improve energy/dEdx calibration. The implementation includes new per-plane TPC normalization tools, updated calibration constants based on Run 2 stopping muons, and infrastructure for MC support via hardcoded run numbers (pending future overlay improvements).
Changes:
- New normalization tools (NormalizeTPCPerPlane, NormalizeTPCPerPlaneSQL) with per-plane TPC equalization
- Updated calibration constants and database tags (TPC_CalibrationTags_Jul2025, TPC_CalibrationTags_Oct2025)
- HitMerger module extended with ChannelROI associations
- Wire-Cell configuration updates for shifted YZ map files and configurable parameters
- Extensive FCL file reorganization (standard/ directory restructured into mc/, data/, overlay/ subdirectories)
- MC normalization support via hardcoded run numbers (Run 1-4) based on fcl-configured run period
- Updated CAF maker workflows with geant4weight additions
- CRT module code formatting improvements
Reviewed changes
Copilot reviewed 90 out of 96 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| ups/product_deps | Updated dependency versions (icarus_data v10_06_02, icarusalg v10_06_00_06, sbncode v10_06_00_06) |
| HitMerger_module.cc | Added ChannelROI-to-Hit associations for normalization tool access |
| wcls-multitpc-sim-drift-simchannel-yzsim-refactored.jsonnet | Updated to use shifted YZ map files and external variables for configuration |
| wcls-multitpc-sim-drift-simchannel-refactored.jsonnet | Added overlay_drifter support and configurable drifter selection |
| sim.jsonnet | Externalized DBFileName and DBTag for runtime configuration |
| detsimmodules_wirecell_ICARUS.fcl | Added YZ map configuration, updated gains, and time offsets for 2D deconvolution |
| normtools_icarus.fcl | Updated to per-plane TPC normalization, added MC flag, new calibration constants |
| calorimetryICARUS.fcl | MC gains now match data gains per updated calibration |
| NormalizeYZSQL_tool.cc | Added MC flag for hardcoded run number lookup |
| NormalizeTPCSQL_tool.cc | Added MC flag and debug output |
| NormalizeTPCPerPlane_tool.cc | New HTTP-based per-plane TPC normalization tool |
| NormalizeTPCPerPlaneSQL_tool.cc | New SQL-based per-plane TPC normalization tool |
| CMakeLists.txt (Calorimetry) | Added build targets for new per-plane normalization tools |
| CRTDataAnalysis_module.cc | Code formatting improvements and added fHitRun/fHitSubRun branches |
| fcl/standard/* | Reorganized into mc/, data/, overlay/ subdirectories with new CMakeLists structure |
| stage1_run2_icarus_MC.fcl | Added normalization tool configuration for MC showers with hardcoded run periods |
| stage1_icarus_defs.fcl | Updated hit merger integration, pulse train hits, and normalization tool setup |
| cafmaker configurations | Added geant4weight to systematic weight sequences |
| calibration_database_*_icarus.fcl | New tag sets for Jul2025 and Oct2025 calibrations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
carriganm95
force-pushed
the
showerNormTools
branch
from
d48aad5 to
0c14fd4
Compare
…alizeTPC->NormalizeTPCPerPlane
|
@carriganm95 and @gputnam Do y'all have reviewers in mind for this PR? I want to assign people to get things rolling before the collaboration meeting rolls around. |
Hi, no I don't have anyone specific in mind. @brucehoward-physics or @cerati know the most about it if they have some time. |
This PR adds the ability to use tools to normalize the shower dEdx and energy using the TPC, YZ, and drift sqlite normalization tools.
For MC only the TPC and YZ normalization tools are used. Additionally, because the maps change based on run and the MC does not have valid run numbers to look up the maps in the sqlite database the run must be hard coded in the fcl files. Based on the run set in the fcl file a run from the correct Run era will be used to obtain the correct maps from the database. In the future overlays should fix the need for these hard coded values.
This PR also depends on a PR to larpandora which adds functions to the energy/dedx tools to run the normalizations.