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

Flight Modes setup should not report incomplete when using Joystick #12415

Open
DonLakeFlyer opened this issue Feb 9, 2025 · 1 comment
Open
Assignees
Milestone

Comments

@DonLakeFlyer
Copy link
Contributor

DonLakeFlyer commented Feb 9, 2025

If you are using a Joystick then you don't set up flight modes on an RC channel. It's super annoying when it keeps telling you flight modes aren't set up. This is actually a Herelink problem in that it supports a joystick, but the joystick sends RC signals.

@DonLakeFlyer DonLakeFlyer added this to the Release V5.0 milestone Feb 9, 2025
@DonLakeFlyer DonLakeFlyer self-assigned this Feb 9, 2025
@DonLakeFlyer DonLakeFlyer changed the title Flight Modes setup should report incomplete when using Joystick Flight Modes setup should not report incomplete when using Joystick Feb 11, 2025
@DonLakeFlyer
Copy link
Contributor Author

@rjehangir, @patrickelectric Hey ArduSub guys.

I'm trying to figure out the usage of supportsRadio which was added ages ago. You can see it here: https://github.com/mavlink/qgroundcontrol/blob/master/src/AutoPilotPlugins/APM/APMAutoPilotPlugin.cc#L77. It's a bit odd in that it gets rid of radio cal and leaves in flight mode setup. Does ArduSub just have a radio which supports channels and so forth but doesn't need calibration?

I also don't understand why it is used here: https://github.com/mavlink/qgroundcontrol/blob/master/src/UI/toolbar/RCRSSIIndicator.qml#L30. That would seem to indicate that an ArduSub although it has supportRadio=false still sends an rcRSSI value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant