Skip to content

improve udp driver based on other driver receiver thread implementati…#592

Open
llanesc wants to merge 5 commits intobitcraze:masterfrom
llanesc:dev-udpdriver
Open

improve udp driver based on other driver receiver thread implementati…#592
llanesc wants to merge 5 commits intobitcraze:masterfrom
llanesc:dev-udpdriver

Conversation

@llanesc
Copy link

@llanesc llanesc commented Feb 14, 2026

Pull request for #590

I have been using this modified udpdriver for CrazySim. The old one was a bit outdated. It basically just includes the receiver thread and has similar features to the other drivers. It also includes a scan feature that scans if a port exists in 19850-19859 for the cfclient.

@ataffanel ataffanel self-requested a review February 16, 2026 10:38
Copy link
Member

@ataffanel ataffanel left a comment

Choose a reason for hiding this comment

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

Looks great. I have a couple of small comment, the most important being that this changes the protocol so we should make sure it is documented.

@llanesc
Copy link
Author

llanesc commented Feb 16, 2026

@ataffanel Do you think I should also add an out_queue? Or is that particular to the radio hardware limitations? I noticed that if the server is lagging because of a slower than real time firmware simulation some packets are lost. This might benefit from the "safe_link" method implemented in the radio.

@ataffanel
Copy link
Member

For network the out queue might not be required as the network stack will be the queue and will likely be more than fast enough. However if you see packet loss it might be required to switch to TCP instead of UDP. I do not think implementing Safelink over UDP would be very efficient since it requires to implement fast ping pong communication.

We have had great success using ZMQ for communicating over network, maybe it would be a better fit for this use-case?

@llanesc
Copy link
Author

llanesc commented Feb 20, 2026

I verified this version works on CrazySim. I spawned 4 crazyflies on 19850-19853 and they all appeared on the cfclient.

crazysim-cfclient.mp4

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.

2 participants

Comments