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

Roland sipos/data move cb registry #121

Merged
merged 6 commits into from
Mar 28, 2024

Conversation

roland-sipos
Copy link
Member

This branch makes it possible to move data between I/O card receivers (e.g.: NICReceiver) through callback invokation, instead of IOManager (that bring an extra data copy of the full stream).
The option is enabled, if the "cb_" prefix is appended for the "raw_input" connection names in the init step of the DataLinkHandler modules.

Dependency PR is in readoutlibs:
DUNE-DAQ/readoutlibs#164

@roland-sipos roland-sipos added the enhancement New feature or request label Mar 27, 2024
@roland-sipos
Copy link
Member Author

System test with 2 runs, first without "cb_" prefix on "raw_input" connections, and the second with applying the prefix, in which case callbacks are used instead of IOManager senders and receivers:

http://np04-srv-017:31023/d/v5_0_0-overview/daq-overview?orgId=1&var-influxdb=ffedc550-6555-4a16-a113-aabf2b980c30&var-postgresql=c7e9e333-25c9-49e7-b209-b9492f70d419&var-partition=np04hd&var-hsi_series=&var-hsi_field=&var-run_time=4003&from=1711530238784&to=1711530904352

@bieryAtFnal bieryAtFnal added the miscellaneous deliverable A change that is/will be part of a release but is not substantial enough to be a daq-deliverable label Mar 27, 2024
@roland-sipos roland-sipos merged commit 3b28c55 into production/v4 Mar 28, 2024
1 check passed
@roland-sipos roland-sipos deleted the roland-sipos/data-move-cb-registry branch March 28, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request miscellaneous deliverable A change that is/will be part of a release but is not substantial enough to be a daq-deliverable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants