Skip to content

Added new blip energy records#174

Open
Jjm321814 wants to merge 1 commit into
developfrom
feature/BlipEnergyDriftCorrection
Open

Added new blip energy records#174
Jjm321814 wants to merge 1 commit into
developfrom
feature/BlipEnergyDriftCorrection

Conversation

@Jjm321814

Copy link
Copy Markdown
Contributor

Adding additional blip energy variables so analyzers have access to a drift corrected energy, as well as an explicitly not drift corrected one.

All drift corrections are anchored to the event timestamp, so the beam blips will come out with the right energy. Any out-of-time blip activity will be mis-corrected, so the noDriftCorrection variables are saved as new variables.

@Jjm321814

Copy link
Copy Markdown
Contributor Author

This PR is part of a set of 4 in sbndcode, sbnobj, sbncode, and sbnanaobj to make this new blip data entry, and share it to CAF level.
Each PR is at
sbndcode: SBNSoftware/sbndcode#941
sbnobj: #174
sbncode: SBNSoftware/sbncode#661
sbnanaobj: SBNSoftware/sbnanaobj#193

@Jjm321814 Jjm321814 changed the title Draft: Added new blip energy records Added new blip energy records Jul 13, 2026
@Jjm321814

Copy link
Copy Markdown
Contributor Author

This PR just adds 3 data members to the larsoft blip class.
These are each explicitly not-drift-corrected, while the regular energy variables become "drift-corrected" (assuming the blip was from the beam).
Happy to change the naming scheme to something shorter if requested.

@Jjm321814

Copy link
Copy Markdown
Contributor Author

I have verified this code imports the correct lifetime values and adds additional variables to the blip larsoft output (reco2) and SRBlip output (CAF).
The validation was done on 10 data events from one run and 10 MC events.

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