Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create AP_RCProtocol::FDM and AP_RCProtocol::UDP_Packed16BitValues #26510

Merged
merged 6 commits into from
Mar 25, 2024

Conversation

peterbarker
Copy link
Contributor

... and use them in SITL.

Do we really need to worry about the hand-over from port-5501 input to FDM so much?

i.e. if you manage to be sending packed values to port 5501 and then connect to the simulator - do you really expect to then get RC input from FlightAxis?

Tested "normal" SITL using sim_vehicle.py.

Tested the FDM input using FlightAxis.

@peterbarker
Copy link
Contributor Author

peterbarker commented Mar 15, 2024

Tested XPlane RC Inputs the same as master. Not quite, RC_Channels now reports correct number of channels:

Master:

RTL> 47: RC_CHANNELS {time_boot_ms : 1174135, chancount : 16, chan1_raw : 650, chan2_raw : 650, chan3_raw : 2333, chan4_raw : 1423, chan5_raw : 1420, chan6_raw : 1000, chan7_raw : 1000, chan8_raw : 1000, chan9_raw : 1000, chan10_raw : 0, chan11_raw : 0, chan12_raw : 0, chan13_raw : 0, chan14_raw : 0, chan15_raw : 0, chan16_raw : 0, chan17_raw : 0, chan18_raw : 0, rssi : 255}

This:

RTL> 46: RC_CHANNELS {time_boot_ms : 1418251, chancount : 9, chan1_raw : 650, chan2_raw : 650, chan3_raw : 2333, chan4_raw : 1423, chan5_raw : 1420, chan6_raw : 1000, chan7_raw : 1000, chan8_raw : 1000, chan9_raw : 1000, chan10_raw : 0, chan11_raw : 0, chan12_raw : 0, chan13_raw : 0, chan14_raw : 0, chan15_raw : 0, chan16_raw : 0, chan17_raw : 0, chan18_raw : 0, rssi : 255}

@peterbarker
Copy link
Contributor Author

Board                    AP_Periph  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
CubeOrange-periph-heavy  *                                                                     
Durandal                            *      *           *       *                 *      *      *
Hitec-Airspeed           *                                                                     
KakuteH7-bdshot                     *      *           *       *                 *      *      *
MatekF405                           *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot                  *                  *       *                 *      *      *
f103-QiotekPeriph        *                                                                     
f303-Universal           *                                                                     
iomcu                                                                *                         
revo-mini                           *      *           *       *                 *      *      *
skyviper-v2450                                         *                                       

Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

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

i'd prefer UDP not whatever the heck this is

@peterbarker peterbarker force-pushed the pr/rcprotocol-udp branch 2 times, most recently from fd7bef5 to ad29cba Compare March 20, 2024 09:59
@peterbarker peterbarker merged commit 964c30e into ArduPilot:master Mar 25, 2024
91 checks passed
@peterbarker peterbarker deleted the pr/rcprotocol-udp branch April 1, 2024 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants