Remove `DetectorSynchronization` - Githubissues

XENONnT / straxen

Streaming analysis for XENON

BSD 3-Clause "New" or "Revised" License

20 stars 32 forks source link

Remove `DetectorSynchronization` #1414

Closed dachengx closed 2 weeks ago

dachengx commented 2 weeks 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?

Because they should already been deleted in https://github.com/XENONnT/straxen/pull/1285

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:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coveralls commented 2 weeks ago

coverage: 90.836% (+0.1%) from 90.739% when pulling 50372673911f9c05d6174c0ca6b65057c2fe9e16 on del_detector_time_offsets into 007ede17dc181ca50ac1d928f2bd20b2b1f748ad on master.