Updated Dead PMT list (data and MC)

[S81D]

Describe your changes

As discussed in PR #357, there were a few PMTs that were dead/inactive in the data which I added to the mask file to omit them for MC. I took a deeper dive (see docdb link below) at the PMT behavior over time (FY22 - FY25), and have since identified more faulty / poor performing PMTs. Those should be masked out from both the data and MC moving forward.

To do this in the data, we can update the FullTankPMTGeometry.csv and set the detector_status to OFF. When the PhaseIIADCHitFinder tool looks for pulses, it fetches the detector status through the DataModel, and ignores PMTs that are designated as OFF. I have also added that check to the ClusterFinder and PhaseIITreeMaker tool (with a configuration to do so) to retroactively apply the mask since the data already processed will not have the updated dead PMT IDs. This allows analyzers flexibility to mask out PMTs depending on the data epoch.

To do this in the MC, we just add the IDs to the dead mask that is read in by LoadWCSim, which will not pass any MC hits from that channel to downstream tools. This is only necessary if you are using the parametric model, as PMTWaveformSim interfaces with the hit finding tools and is done the same as the data.

Checklist before submitting your PR

• This PR implements a single change (one new/modified Tool, or a set of changes to implement one new/modified feature)
• This PR alters the minimum number of files to affect this change
• [N/A] If this PR includes a new Tool, a README and minimal demonstration ToolChain is provided
• [N/A] If a new Tool/ToolChain requires model or configuration files, their paths are not hard-coded, and means of generating those files is described in the readme, with examples provided on /pnfs/annie/persistent
• [N/A] For every new usage, there is a reason the data must be on the heap
• [N/A] For every new there is a delete, unless I explicitly know why (e.g. ROOT or a BoostStore takes ownership)

Additional Material

Please refer to these slides for more details.

[jminock]

Looks good!

[S81D]

There's just one more change i would like to make to the TreeMaker - currently it is not masking out the clustered hitT, hitPE, etc... objects for the dead PMTs. It is properly omitting them from the cluster parameters (charge, time, charge balance, etc...) and omitting those hits from the trigger tree, but there is an extra condition needed for the hits in the cluster tree. Should have that change soon. For now its on hold.

[S81D]

alright all good.

[marc1uk]

goto: ...!
.... i'll allow it.