Skip to content
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

Update ClusterFinderConfig - expand MC pulse width #322

Open
wants to merge 4 commits into
base: Application
Choose a base branch
from

Conversation

S81D
Copy link
Contributor

@S81D S81D commented Dec 5, 2024

Describe your changes

Extend MC pulse width to align data and MC. Photons hit within this larger window will be considered a "hit". 10ns is too narrow and leads to WAY too many PMT "hits" for high energy events. 35ns seems to align the number of PMT hits between data and MC.

As there are still a few things that need to be optimized for the tuning campaign, this may change in the future. Hit digitization seems like it will be changed in way of a Waveform simulator that Andrew is planning on rolling in. Therefore the pulse width may have to be changed in the future. This is a "best fit" to what we currently have.

I included the PR as James is beginning his production run for the WCSim / ToolAnalysis analysis root files.

Checklist before submitting your PR

  • [ x] This PR implements a single change (one new/modified Tool, or a set of changes to implement one new/modified feature)
  • [ x] 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

Slide 9 from this presentation highlights the change: https://annie-docdb.fnal.gov/cgi-bin/sso/RetrieveFile?docid=5488&filename=WCSim%20Tuning%20Status%20_%20June%203%202024%20_%20Steven%20Doran%20_%20ANNIE%20summer%20workshop%202024.pdf&version=2

Again there's more things to re-tune i just wanted to get this on board as James is about to produce some collaboration-wide files.

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.

1 participant