[Bug] uXRCE-DDS RTT TOO HIGH, FIXED uORB INTERVAL - Githubissues

PX4 / PX4-Autopilot

PX4 Autopilot Software

https://px4.io

BSD 3-Clause "New" or "Revised" License

8.2k stars 13.37k forks source link

[Bug] uXRCE-DDS RTT TOO HIGH, FIXED uORB INTERVAL #22286

Open zauberflote1 opened 10 months ago

zauberflote1 commented 10 months ago

Describe the bug

Fixed uORB interval for uXRCE-DDS topics https://github.com/PX4/PX4-Autopilot/blob/main/src/modules/uxrce_dds_client/dds_topics.h.em#L32C1-L33C1

We might want to remove uORB interval. RTT is too high.

To Reproduce

Run HITL or Flight Test and check timesync_status topic

Expected behavior

Increased RTT

Screenshot / Media

No response

Flight Log

https://logs.px4.io/plot_app?log=423f8d60-2b27-48c9-b7e0-e2d3252f6e08

Software Version

Main

Flight controller

PX4FMU-V5

Vehicle type

Multicopter

How are the different components wired up (including port information)

HITL, TELEM2 - Serial Companion Computer MicroUSB - Serial Host Computer

Additional context

No response

beniaminopozzan commented 10 months ago

Hi @Zauberflote1 , I compared v1.14.0, main and main with UXRCE_DEFAULT_POLL_RATE=1 (note that the naming is wrong, that symbol defines the minimum interval in milliseconds) and in all three cases the rtt is below 1ms.