Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementation of TRestRawSignalRecoverSaturationProcess #147

Merged
merged 18 commits into from
Feb 10, 2025

Conversation

AlvaroEzq
Copy link
Contributor

@AlvaroEzq AlvaroEzq commented Nov 11, 2024

AlvaroEzq Large: 522 Powered by Pull Request Badge

Implementation of a new TRestEventProcess that aims to recover saturated signals. In order to recover it, the idea is to fit the signal (ignoring the saturated part of it) and reconstruct the saturated bins by replacing them by the value of the fitted function on those time bins. This will allow to recover the peak position and amplitude needed in further analysys processes.

It produces the following observables:

  • addedAmplitude: sum of the extra amplitude (newamplitude - previousamplitude) of the saturated signals of the event
  • addedIntegral: sum of the extra integral (sum of newvalue-previousvalue of the saturated bins) of the saturated signals of the event
  • saturatedSignals: number of signals of the event detected as saturated
  • recoveredSignals: number of signals of the event detected as saturated and that have been modified by the fitted pulse. It is always <=saturatedSignals. If it is less, it means that the fit values of the saturated bins of the signal were lower than the original value of the signal, so they are not replaced.

@AlvaroEzq AlvaroEzq force-pushed the alvaroezq_recoverSat branch from f11871d to 211c55f Compare February 5, 2025 13:11
@AlvaroEzq AlvaroEzq changed the title Implementation of a process to recover saturated signals Implementation of TRestRawSignalRecoverSaturationProcess Feb 10, 2025
@AlvaroEzq AlvaroEzq requested a review from JPorron February 10, 2025 10:53
@AlvaroEzq AlvaroEzq merged commit 12d3311 into master Feb 10, 2025
64 checks passed
@AlvaroEzq AlvaroEzq deleted the alvaroezq_recoverSat branch February 10, 2025 11:35
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.

2 participants