Feature/blip energy drift correction#941
Conversation
|
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. |
|
Initial checks on MC looked good. The values of energy were very mildly shifted with temporary debug print outs like Unfortunately the detector properties service used to get the electron lifetime initially applies a fixed lifetime. We will need to connect to some kind of calibration database service to get better results on data. |
…ol easier to import
|
This PR is more complicated. This PR does not interrupt PANDORA's use of the NormalizeDriftSQLite_tool and allows blips to use NormalizeDriftSQLite_class. If we have an example of implementation of the art_tool version (that doesn't result in the code duplication seen in sbndcode/sbndcode/Calorimetry/LightCaloProducer_module.cc Lines 821 to 848 in 27d88f5 |
|
Oh oops let me get rid of the print statements I used to check the implementation. |
|
I have verified this code imports the correct lifetime values and adds additional variables to the blip larsoft output (reco2) and SRBlip output (CAF). |
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.