Skip to content

CRT topology change#32

Closed
denizergonul wants to merge 11 commits intodevelopfrom
dte/crt_topology_change
Closed

CRT topology change#32
denizergonul wants to merge 11 commits intodevelopfrom
dte/crt_topology_change

Conversation

@denizergonul
Copy link
Copy Markdown
Contributor

@denizergonul denizergonul commented Mar 6, 2026

  • CRT frame builders:
    • Communicate with SocketWriter via IOM.
    • Have a dedicated D2D connection (with 1 of the DataSenders).
    • Spawn producer threads for each DetectorStream.
    • Have 1 MPMC (practically, MPSC) IOM sender.

CRT(Bern/Grenoble)ReaderModule currently generates fake packets (to be replaced by real CRT hardware in the future).

  1. Map source IDs to stream IDs (m_fake_stream_ids).
    e.g.
    1007->0
    1008->1
  2. Start a producer thread that, for each stream:
  • Generates a fake packet with proper stream ID.
  • Sends packet via the queue.
    e.g.
    packet.stream_id=0 uses IOM1
    packet.stream_id=1 uses IOM1

appmodel PR dte/crt_topology_change
daqsystemtest PR dte/crt_topology_change
crtmodules PR dte/crt_topology_change
asiolibs PR dte/crt_topology_change
daqconf PR dte/crt_cb_update
datahandlinglibs PR dte/cb_acquire
fdreadoutmodules PR dte/crt_rates

CRT readers communicate with socket writers via IOM.
Socket readers communicate with DHLs via callbacks.

To try out:
pytest -s integtest/crt_frame_buider_test.py


Type of change

  • New feature or enhancement (non-breaking change which adds functionality)

Testing checklist

  • Python tests pass if applicable (e.g. python -m pytest)

Further checks

Copy link
Copy Markdown

@alessandrothea alessandrothea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wesketchum wesketchum mentioned this pull request Mar 23, 2026
@bieryAtFnal
Copy link
Copy Markdown
Contributor

Closing this PR since the branch has been merged to the prep-release/fddaq-v5.6.0 branch in a different PR.

@bieryAtFnal bieryAtFnal deleted the dte/crt_topology_change branch March 24, 2026 16:00
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.

4 participants