search

darkmatter-ucsd / SanDP

SanDix Processor
0 stars 1 forks source link

Inconsistent S1/S2 width definiton #10

Closed jingqiangye closed 5 years ago

jingqiangye commented 5 years ago

S1/S2 width requirements are different stages are inconsistent.

In the very first step, when sandp tries to find potential peaks, the width definition is simply the width of smoothed data that is above threshold, see here

But later when sandp merges peaks before main s2, and tell if the merged peaks are s1 or s2, it uses the width of 50% height, see here. In this case, merged s2 must have a quite wide width to be identified as s2. As a result, some small s2, especially single electron s2s will be misidentified as S1.

jingqiangye commented 5 years ago

Sorry it's the same definition, see here. 50% width is only used for defining rise time.