Skip to content

BeamFetcherV2 tool changes #273

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.

Sign up for GitHub

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

Jump to bottom
Merged
marc1uk merged 3 commits into from
Aug 16, 2024
Merged

BeamFetcherV2 tool changes #273

marc1uk merged 3 commits into from
Aug 16, 2024

Conversation

@S81D
Copy link
Collaborator

@S81D S81D commented Aug 15, 2024

Rolled in some changes from Andrew and some other modifications:

  • Some bug fixes, primarily where the tool misses some CTC entries and doesn't pass them along to other tools or saves them into the output beam root file
  • Yue's new event builder is based on the undelayed beam trigger (14), so this tool now looks for CTC triggers to pair with fetched beam information based on 14 instead of trigger word 5. In practice this has very little effect on the total number of entries or the recorded POT (< 1% difference between the trigger words)

S81D added 3 commits August 12, 2024 14:57
@S81D
Update BeamFetcherV2.cpp
50db1a9 
- fixed a bug where not all timestamps were being saved to the output ROOT file
- clear beamdatamap after writing it to the root file. This increases the speed of the tool by ~2x
- change the beam triggerword from 5 to 14 (undelayed beam trigger). This doesn't really affect the entries we get (< 1% difference in total counts and total recorded POT), but Yue's new event building (coming soon) builds on 14, so trying to stay consistent with that
@S81D
Update BeamFetcherV2.cpp
0e9d24a 
Missed two small changes
@S81D
Update BeamFetcherV2.h
32b50a7
@marc1uk marc1uk merged commit 53828d0 into ANNIEsoft:Application Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@S81D @marc1uk