Skip to content

Update PhaseIIADCHitFinderConfig to match Johann's NoDoubleHits Pulse Finding - #395

Open
anuj-guptta wants to merge 1 commit into
ANNIEsoft:Applicationfrom
anuj-guptta:NoDoubleHits
Open

anuj-guptta wants to merge 1 commit into
ANNIEsoft:Applicationfrom
anuj-guptta:NoDoubleHits

Conversation

@anuj-guptta

@anuj-guptta anuj-guptta commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Following Johann's PR on updated Pulse finding method "NoDoubleHits", I have updated EventbuilderV2*/PhaseIIADCHitFinderConfig and PMTWaveformSim/PhaseIIADCHitFinderConfig to match the updated version.

Describe your changes

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
  • If this PR includes a new Tool, a README and minimal demonstration ToolChain is provided
  • 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
  • For every new usage, there is a reason the data must be on the heap
  • For every new there is a delete, unless I explicitly know why (e.g. ROOT or a BoostStore takes ownership)

Additional Material

Attach any validation or demonstration files here. You may also link to relavant docdb articles.

This branch has not been deployed

No deployments
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