Skip to content

New calorimeter brancher#362

Merged
AndrewEdmonds11 merged 10 commits intoMu2e:mainfrom
sophiemiddleton:fcls
Apr 22, 2026
Merged

New calorimeter brancher#362
AndrewEdmonds11 merged 10 commits intoMu2e:mainfrom
sophiemiddleton:fcls

Conversation

@sophiemiddleton
Copy link
Copy Markdown
Collaborator

This pull request introduces support for saving Monte Carlo (MC) truth information associated with calorimeter digis into the EventNtuple. The main changes include the addition of a new CaloDigiMCInfo struct, updates to the configuration and data handling in EventNtupleMaker_module.cc, and the implementation of helper methods to fill the new MC info. These changes allow for the optional storage of detailed MC truth for calorimeter digis and their associated simulation particles.

New data structure for Calorimeter Digi MC information:

  • Added CaloDigiMCInfo struct to hold MC-truth information for calorimeter digis, including deposited energies, times, sim particle IDs, and relationships. (inc/CaloDigiMCInfo.hh, src/classes.h, src/classes_def.xml) [1] [2] [3]

EventNtupleMaker module enhancements:

  • Added configuration options and data members to enable writing out calorimeter digi MC information and associated sim info. (src/EventNtupleMaker_module.cc) [1] [2] [3] [4]
  • Updated the module to create, clear, and fill new branches for CaloDigiMCInfo and associated sim info, including retrieval of the necessary input collections. (src/EventNtupleMaker_module.cc) [1] [2] [3] [4]

Helper methods for filling MC info:

  • Implemented fillCaloDigiMCInfo and fillCaloDigiSimInfos in InfoMCStructHelper to populate the new data structures from CaloShowerSim objects. (src/InfoMCStructHelper.cc, inc/InfoMCStructHelper.hh) [1] [2] [3]

Configuration and FCL updates:

  • Updated FCL configuration to support toggling the new MC branches for calorimeter digis and to specify the relevant input tags. (fcl/from_dig-calo.fcl)

These changes collectively provide the infrastructure to store and access detailed MC truth for calorimeter digis in the EventNtuple, supporting future analysis and validation efforts.

@sophiemiddleton sophiemiddleton changed the title Fcls New calorimeter brancher Apr 16, 2026
Copy link
Copy Markdown
Collaborator

@AndrewEdmonds11 AndrewEdmonds11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this, Sophie! Could you also add the new branches to the ntuplehelper following Step 8 of these instructions? https://github.com/Mu2e/EventNtuple/blob/main/doc/developers.md#Adding-a-branch-to-EventNtuple

We can sort out RooUtil and validation (Step 9) later

Comment thread doc/branches.md

| branch | structure | explanation | key fields | leaf information |
|--------|-----------|-------------|-----------|------------------|
| caloclustersmc | Vector branch | MC-truth information for calorimeter clusters (aligned with caloclusters)| `nsim` (# of sim particles), `etot` (total true energy), `tavg` (avg time), `eprimary` (primary particle energy), `tprimary` (primary time), `simParticleIds`, `simRels` (MCRelationship), `hits_`, `prel` (primary to event primary relationship) | [see CaloClusterInfoMC.hh](../inc/CaloClusterInfoMC.hh)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the new calodigimc branch not be here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure I will add it. I think I got interuppted with something else, and I did actually write somethings I didnt push. Will do this today

@sophiemiddleton
Copy link
Copy Markdown
Collaborator Author

I have updated the documentation and the ntuple helper. One thing I noticed, we dont have a calorecodigimc. But it would be exactly like the one for calodigimc, so I didnt make it.

@AndrewEdmonds11 AndrewEdmonds11 merged commit 632b6c5 into Mu2e:main Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants