strax icon indicating copy to clipboard operation
strax copied to clipboard

Lone hit area

Open shenyangshi opened this issue 1 year ago • 4 comments

https://github.com/AxFoundation/strax/blob/afaf8cf6e468a87977eec8a6cb76812ada175437/strax/processing/peak_building.py#L482-L484

If we make the lone hits area also normalized according to PMT adc_to_pe like the peaklets instead of [ADC x samples], will it cause trouble elsewhere?

shenyangshi avatar Jun 07 '23 14:06 shenyangshi

We would need to check where it is called. Would there be a big benefit changing it into pe?

WenzDaniel avatar Jun 08 '23 15:06 WenzDaniel

Thanks for the quick response @WenzDaniel! Since the gain model is a time-dependent correction, people interested in lone hits properties like area, DPE component or stability need to fetch the corrections themselves and correct run-by-run. I think it's nice if we can do this at the basement level.

shenyangshi avatar Jun 08 '23 15:06 shenyangshi

Maybe @a1exndr and @paloma can comment on the influence on the lone hit gain correction.

shenyangshi avatar Jun 08 '23 15:06 shenyangshi

We can change it if needed. But this requires some careful checks and testing.

WenzDaniel avatar Jun 08 '23 15:06 WenzDaniel