XENONnT / straxen

Streaming analysis for XENON
BSD 3-Clause "New" or "Revised" License
20 stars 32 forks source link

Remove unnecessary check in `merged_s2s` #1195

Closed dachengx closed 1 year ago

dachengx commented 1 year ago

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible

What does the code in this PR do / what does it improve?

Sometimes end_merge_at and start_merge_at can be empty arrays. Function _filter_s1_starts has already guaranteed that the plugin will not merge a single peak.

Can you briefly describe how it works?

Can you give a minimal working example (or illustrate with a figure)?

Please include the following if applicable:

Notes on testing

All italic comments can be removed from this template.

coveralls commented 1 year ago

Coverage Status

coverage: 93.439% (-0.01%) from 93.451% when pulling f25a0e4c127ecabb7300ec0cbb9b2d86e51f34ec on rm_merged_s2s_check into 40f75b192a5dfbe81a2220e168086eb307cc1a8b on master.