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

Autopilot capabilities #2181

Merged
merged 5 commits into from
May 23, 2015
Merged

Autopilot capabilities #2181

merged 5 commits into from
May 23, 2015

Conversation

RomanBapst
Copy link
Contributor

Send this message on startup and on request. Corresponding issue is #1861

@RomanBapst
Copy link
Contributor Author

@LorenzMeier Comments?

@LorenzMeier
Copy link
Member

Looks good so far, now we just need to populate the bitmask correctly.

@LorenzMeier
Copy link
Member

Do you have a WIP branch which fills the static part of the bit mask already?

@LorenzMeier
Copy link
Member

@Tumbili do you have a branch / commits implementing the rest of it somewhere? If not no worries and I'll finish this up tonight so we can close this as well.

@TSC21
Copy link
Member

TSC21 commented May 22, 2015

mavros is waiting for this message support also: mavlink/mavros#246 as mavlink/mavros#96 is already implemented.

@LorenzMeier LorenzMeier force-pushed the autopilot_capabilities branch 2 times, most recently from 8561be4 to 0b5480f Compare May 22, 2015 20:23
@LorenzMeier
Copy link
Member

@Tumbili @kd0aij @mcharleb Implemented git version now as file as suggested. Please test.

@LorenzMeier LorenzMeier merged commit ccac324 into master May 23, 2015
@LorenzMeier LorenzMeier deleted the autopilot_capabilities branch May 23, 2015 09:07
@LorenzMeier
Copy link
Member

@TSC21 Its in master and the release branches now.

@TSC21
Copy link
Member

TSC21 commented May 23, 2015

@LorenzMeier thank you for the feedback. I already tested it on mavros side and I got this:

[ INFO] [1432376904.331698271]: VER: Capabilities 0x00000000000004eb
[ INFO] [1432376904.331779081]: VER: Flight software:     00000000 (j�?#Q6I`j�?#Q6I`)
[ INFO] [1432376904.331814379]: VER: Middleware software: 00000000 (j�?#Q6I`)
[ INFO] [1432376904.331850687]: VER: OS software:         00000000 (        )
[ INFO] [1432376904.331881663]: VER: Board hardware:      00000000
[ INFO] [1432376904.331912500]: VER: VID/PID: 26ac:0011
[ INFO] [1432376904.331939803]: VER: UID: 3432470a31323533

I can't really tell if this is a problem on mavros side or in the firmware side, or if this values represent a problem at all.

@LorenzMeier LorenzMeier restored the autopilot_capabilities branch June 12, 2015 06:58
@LorenzMeier LorenzMeier deleted the autopilot_capabilities branch June 28, 2016 20:26
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.

3 participants