-
Notifications
You must be signed in to change notification settings - Fork 3
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
Reorganize daqconf schema options #345
Closed
eflumerf opened this issue
Jun 1, 2023
· 3 comments
· Fixed by #354, DUNE-DAQ/flxlibs#120, DUNE-DAQ/wibmod#39, DUNE-DAQ/ctbmodules#20 or DUNE-DAQ/timinglibs#92
Closed
Reorganize daqconf schema options #345
eflumerf opened this issue
Jun 1, 2023
· 3 comments
· Fixed by #354, DUNE-DAQ/flxlibs#120, DUNE-DAQ/wibmod#39, DUNE-DAQ/ctbmodules#20 or DUNE-DAQ/timinglibs#92
Labels
enhancement
New feature or request
Comments
I see this issue as a precursor to #330, as better organization will make those changes more straightforward |
PR #354 supersedes #330.
|
I think we'll also need timinglibs, I can do that on Monday. |
This was
linked to
pull requests
Jun 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The daqconf schema (https://github.com/DUNE-DAQ/daqconf/blob/develop/schema/daqconf/confgen.jsonnet) was originally split into groups in a fairly arbitrary way. The parameters should be reorganized to ensure that app-specific parameters are grouped together, and possibly add additional groups where needed for cross-app parameters (instead of putting most of them in
boot
).The text was updated successfully, but these errors were encountered: