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

Refactor bootstrax #479

Open
1 task done
JoranAngevaare opened this issue May 7, 2021 · 2 comments · Fixed by #481
Open
1 task done

Refactor bootstrax #479

JoranAngevaare opened this issue May 7, 2021 · 2 comments · Fixed by #481
Labels
documentation Something should be addressed in the docs enhancement New feature or request

Comments

@JoranAngevaare
Copy link
Contributor

JoranAngevaare commented May 7, 2021

Bootstrax has become a huge script with many nuts and bolts that could do with some refactoring for readability. Since I am not leaving any time soon, this issue can stay open for a while to collect some ideas (feel free to add).

Classes (add the following as objects in bootstrax):

name current # lines Envisaged usage
Process 500 lines For a given run, do run strax multiprocessing
Target 120 lines Given a rundoc, infer which target we should be using (perhaps join with Resources/compression class)
Resources/compression 150 lines Given a rundoc, figure out what cores, max-messages and raw-record compressor can be used for this run
Database class ?500 lines? Communicate with the RunDB/DAQDB about the run, files, host etc.
Logging class .. just use the daq-logger
Host class 300 lines monitor data and state of the host
Config handler ~150 lines Put the configs in a seperate file

Documentation:

  • On the website ~80 lines

Functionality:

  • Smarter shutdown/restart
@JoranAngevaare JoranAngevaare self-assigned this May 7, 2021
@JoranAngevaare JoranAngevaare added documentation Something should be addressed in the docs enhancement New feature or request labels May 7, 2021
@JoranAngevaare JoranAngevaare linked a pull request May 7, 2021 that will close this issue
@JoranAngevaare
Copy link
Contributor Author

Also include XENONnT/utilix#51

@JoranAngevaare
Copy link
Contributor Author

fail when a post-process target is not in the context

@JoranAngevaare JoranAngevaare mentioned this issue Aug 21, 2022
6 tasks
@JoranAngevaare JoranAngevaare removed their assignment May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Something should be addressed in the docs enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant