No need to set `loop_over` for `EventBasics` - Githubissues

XENONnT / straxen

Streaming analysis for XENON

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

19 stars 33 forks source link

No need to set `loop_over` for `EventBasics` #1377

Closed dachengx closed 2 months ago

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

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

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

coverage: 91.526% (+0.009%) from 91.517% when pulling 9b74a4287ffb7b4e06e27c61030ec35eea34c6c4 on event_basics_no_loop_over into a23283c5e4ef99be286fa55aad02570d8477390d on master.