fix pax bug for very rare event

[feigaodm]

This PR tries to fix a bug reported by @ershockley when processing Rn220 calibration data with pax_v6.8.0. This bug is due to de-saturation plugin as this event has very wired saturation. Since not a lot channel are detecting signals during saturation, we should skip this event for de-saturation correction. This will fix issue (#599).

Waveform of this events is here.

[sanderbreur]

After checking it I think we are adding a needed check, but losing another needed check. Because iether the pulse itself or the sumw can have not enough samples. So I would keep this change, but add the deleted part back into the code. (so do it both for w and for sumw)

This should be added back:

•        if len(w[reference_slice][w[reference_slice] > 1]) < self.config['reference_region_samples_treshold']:
  

•            # the pulse is saturated, but there are not enough reference samples to get a good ratio
  

•            # This actually distinguished between S1 and S2 and will only correct S2 signals
  

•            continue
  

[feigaodm]

Thanks @sanderbreur , just added it back and merged the PR.

[sanderbreur]

Perfect, ty Op wo 6 sep. 2017 om 00:21 schreef Fei Gao <[email protected]>:

…

Merged #601 <#601>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#601 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGNstRpkLG3sxYAGyQoYzDKJ4L4EvK1wks5sfcmHgaJpZM4O_GDz> .