XENONnT / straxen

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

Load whole run for `VetoIntervals` regardless the run length #1399

Closed dachengx closed 3 months ago

dachengx commented 3 months 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

Depends on https://github.com/AxFoundation/strax/pull/835 and https://github.com/AxFoundation/strax/pull/855

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

The max_veto_window and veto_proximity_window were introduced by https://github.com/XENONnT/straxen/pull/929. They are assuming the run length is no longer than 1 or 2 hrs, but sometimes the run length can be very large so we might lose catching of the busy time of the DAQ, if using the previous configurations.

Can you briefly describe how it works?

Use ExhaustPlugin to load the dependency of the whole run of all dependencies at once. In this way we do not need to consider the overlapping windows.

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 3 months ago

Coverage Status

coverage: 91.156% (+0.02%) from 91.141% when pulling 316020a2ebf956d70bfb38762d0ffd465974ecd4 on long_veto_intervals into 58a78b7068d293c8ec1fa275a2495b486020927b on master.