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

ArduPlane v3.7.0 - Sends mavlink messages with two different system ids #4912

Closed
DonLakeFlyer opened this issue Sep 30, 2016 · 8 comments
Closed
Assignees
Milestone

Comments

@DonLakeFlyer
Copy link
Contributor

Issue details

Steps:

  • Flash ArduPlane 3.7.0 to Pixhawk
  • Boot QGC (with Pixhawk disconnected)
  • Connect Pixhawk to QGC via USB

Result:
ArduPlane sends mavlink messages with sysid of 7 and 1. 1 is the correct system id. This causes QGC to think there are two different vehicles connected and shows all it's multi-vehicle ui. Pretty bad bug for any groundstation that supports multiple vehicles. The mavlink messages with id 7 are sent first during board boot I believe, then it switches to 1.

Here is QGC mavlink inspector showing the messages:
screen shot 2016-09-30 at 3 57 48 pm

@DonLakeFlyer
Copy link
Contributor Author

ArduCopter 3.3 doesn't do this. ArduPlane didn't used to do this in older versions.

@mboland
Copy link

mboland commented Sep 30, 2016

Arducopter does this in 3.4, QGroundControl and Mission Planner

@tridge tridge self-assigned this Oct 1, 2016
@tridge
Copy link
Contributor

tridge commented Oct 1, 2016

thanks for letting me know, I'll fix this now

@CraigElder CraigElder added this to the Plane v3.7.1 milestone Oct 4, 2016
@rmackay9 rmackay9 modified the milestones: AC 3.4.0, Plane v3.7.1 Oct 6, 2016
@rmackay9
Copy link
Contributor

rmackay9 commented Oct 6, 2016

I tried to set this to Copter-3.4 milestone but github doesn't accept two milestones so resetting to plane only.

@KIrill-ka
Copy link

As a workaround, I set MAV sysid to 7 :)

@DonLakeFlyer
Copy link
Contributor Author

Ah, thanks for the suggestion! Helpful for my testing.

@OXINARF
Copy link
Member

OXINARF commented Oct 18, 2016

Fixed in master.

@OXINARF OXINARF closed this as completed Oct 18, 2016
@rmackay9
Copy link
Contributor

Oh, excellent that it's fixed!

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

No branches or pull requests

7 participants