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

mavlink properly wrap heading fields #9905

Merged
merged 2 commits into from Jul 13, 2018

Conversation

Projects
None yet
2 participants
@dagar
Copy link
Member

dagar commented Jul 13, 2018

These fields are unsigned 0 to 360 degrees, so the yaw estimate in radians need to be wrapped 0 to 2pi first, then converted to degrees, and then sometimes scaled.

dagar added some commits Jul 13, 2018

@dagar

This comment has been minimized.

Copy link
Member Author

dagar commented Jul 13, 2018

Need to check if ADSB_VEHICLE heading is correct.

@LorenzMeier LorenzMeier merged commit 1ea63e4 into master Jul 13, 2018

2 checks passed

WIP ready for review
Details
continuous-integration/jenkins/branch This commit looks good
Details

@LorenzMeier LorenzMeier deleted the pr-mavlink_hdg branch Jul 13, 2018

@dagar dagar added the bug label Jul 13, 2018

@dagar dagar added this to the Release v1.8.1 milestone Jul 13, 2018

@dagar dagar referenced this pull request Jul 27, 2018

Closed

PX4 Stable v1.8.1 Planning #10064

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