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

AssignBunchTimingMC tool + config file changes #323

Merged
merged 8 commits into from
Dec 16, 2024

Conversation

S81D
Copy link
Contributor

@S81D S81D commented Dec 5, 2024

Describe your changes

James processed new tank samples that have smaller flux windows and utilize a different timing scheme, where t0 is assigned as the beam dump rather than when the particle enters the detector geometry. The AssignBunchTimingMC tool has a hard coded configuration that uses the average neutrino flight time from beam dump to the detector (extracted from all WCSim production samples) and uses that to construct BNB-style bunches. As we are using a new flux window (size reduced by x2), the flight time needs to be adjusted.

Also added the configuration file for this tool to some popular toolchains (BeamClusterAnalysisMC and CC_MC_Reco_ntuple).

Added additional configuration variables to the PhaseIITreeMaker (including one for the Bunch Timing tool) that are absent in the normal configuration variables within the toolchains mentioned above.

This PR is mainly to get all configurations and tools in order for James' end-to-end CC analysis workflow. It involves more than a single change but its just adding the same config file over and over for different toolchains.

Checklist before submitting your PR

  • [ see note above - multiple configs are added but its for a single purpose ] 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
  • [ X ] 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

Details on how the bunch timing is assigned can be found here: https://annie-docdb.fnal.gov/cgi-bin/sso/RetrieveFile?docid=5768&filename=AssignBunchTimingMC%20Tool%20PR%20304%20_%20Steven%20Doran.pdf&version=1

S81D added 8 commits December 5, 2024 15:06
new neutrino propagation time to the tank, from James' new tank samples with a narrowed flux window.
Add this tool (and its config) to James' CC analysis toolchain.
Add it to the MC BeamCluster toolchain too
Add config variables that the tool asks for. Tank clustering, MRD clustering, and trigger clustering are all default 1 but best to add it in. Also added the AssignBunchTiming
@marc1uk marc1uk merged commit acd73f3 into ANNIEsoft:Application Dec 16, 2024
1 check passed
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.

2 participants