From bcd257e90f72964c72afe420d8566bdfdb1f96ce Mon Sep 17 00:00:00 2001 From: Dhanya Thattil Date: Mon, 6 Jul 2026 14:15:54 +0200 Subject: [PATCH] minor --- RELEASE.md | 2 ++ slsReceiverSoftware/src/ClientInterface.cpp | 4 ++-- slsReceiverSoftware/src/Implementation.cpp | 4 ++-- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/RELEASE.md b/RELEASE.md index 0209a314a..11563b7af 100644 --- a/RELEASE.md +++ b/RELEASE.md @@ -69,6 +69,8 @@ added rx_streamdummyheader to send the zmq dummy header any time. Allows pre-con allow disabling one UDP interface in the receiver. +setting number of UDP interfaces can only be set at detector level and not at module level. (individual modules) + 2 On-board Detector Server Compatibility ========================================== diff --git a/slsReceiverSoftware/src/ClientInterface.cpp b/slsReceiverSoftware/src/ClientInterface.cpp index b6e3253f9..668f94e55 100644 --- a/slsReceiverSoftware/src/ClientInterface.cpp +++ b/slsReceiverSoftware/src/ClientInterface.cpp @@ -1681,6 +1681,8 @@ void ClientInterface::validate_port_position(const portPosition port) { } int ClientInterface::set_port_udp_datastream(Interface &socket) { + if (detType != EIGER && detType != JUNGFRAU && detType != MOENCH) + functionNotImplemented(); int args[2]{-1, -1}; socket.Receive(args); portPosition port = static_cast(args[0]); @@ -1692,8 +1694,6 @@ int ClientInterface::set_port_udp_datastream(Interface &socket) { LOG(logDEBUG1) << "Setting udp datastream (" << ToString(port) << ") to " << ToString(enable); verifyIdle(socket); - if (detType != EIGER && detType != JUNGFRAU && detType != MOENCH) - functionNotImplemented(); impl()->setUDPDataStream(port, enable); return socket.Send(OK); } diff --git a/slsReceiverSoftware/src/Implementation.cpp b/slsReceiverSoftware/src/Implementation.cpp index 1c2649d66..a8c2d2213 100644 --- a/slsReceiverSoftware/src/Implementation.cpp +++ b/slsReceiverSoftware/src/Implementation.cpp @@ -1626,8 +1626,8 @@ void Implementation::setTenGigaEnable(const bool b) { // datastream can be disabled/enabled only for Eiger 10GbE if (generalData->detType == EIGER) { if (!b) { - udpDataStream[LEFT] = 1; - udpDataStream[RIGHT] = 1; + udpDataStream[LEFT] = true; + udpDataStream[RIGHT] = true; } else { udpDataStream[LEFT] = udpDataStream10GbE[LEFT]; udpDataStream[RIGHT] = udpDataStream10GbE[RIGHT];