Skip to content

Modified the socket_reader_test so that it uses a 4x longer readout window…#39

Open
bieryAtFnal wants to merge 2 commits intodte/crt_topology_changefrom
kbiery/integtest_small_mods
Open

Modified the socket_reader_test so that it uses a 4x longer readout window…#39
bieryAtFnal wants to merge 2 commits intodte/crt_topology_changefrom
kbiery/integtest_small_mods

Conversation

@bieryAtFnal
Copy link
Contributor

…in response to a longer time_tick_difference being used for CRT data packets. Also added some HSI fragment validity checking and an expected TriggerRecord count (to help identify future odd behaviors). Also replaced 'nanorc' with 'dunerc', as has been done elsewhere.

Description

Please note that the target branch for these changes is dte/crt_topology_change.

These changes are primarily to get this integtest working with the latest 'crt topology' changes.

To test these changes, I suggest using a software area based on a recently nightly and adding the following repos with the specified branches.

appmodel: dte/crt_topology_change [d344896]
asiolibs: kbiery/integtest_small_mods [472cd32]
crtmodules: dte/crt_topology_change* [815c702]
daqconf: dte/crt_cb_update [79c5f71]
daqsystemtest: dte/crt_topology_change* [b125aeb]
datahandlinglibs: dte/cb_acquire [52010b5]
dfmodules: develop [c1af62d]
fddetdataformats: develop [7c2117d]
fdreadoutlibs: develop [1d9753c]
fdreadoutmodules: kbiery/crt_fake_param_mods [8bd85fe]
hsilibs: develop [21d3b82]
trigger: develop [3b5e6dc]
utilities: develop [65a37c3]

With these branches, I see that the asiolibs/socket_reader_test and the crtmodules/crt_reader_test run successfully.

Type of change

  • Optimization (non-breaking change that improves code/performance)

Testing checklist

  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)

Kurt Biery and others added 2 commits March 18, 2026 16:17
…indow, in response to a longer time_tick_difference being used for CRT data packets. Also added some HSI fragment validity checking and an expected TriggerRecord count (to help identify future odd behaviors). Also replaced 'nanorc' with 'dunerc', as has been done elsewhere.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants