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

Make the OSD home arrow closer to the actual home direction #3387

Merged
merged 1 commit into from Jun 22, 2018

Conversation

Projects
None yet
1 participant
@fiam
Copy link
Member

commented Jun 18, 2018

We were using the first image (up) for [0deg, 22deg] which made
some directions like [350deg, 360deg) feel pretty unnatural because
the craft was almost pointing home but the arrow pointed a bit to
the left. Maximum error between actual direction and the arrow was
22deg.

With this change, the up image is used for [349deg, 11deg], 2nd
image covers [12deg, 33deg], etc... which makes the error between
the actual direction and the arrow smaller. This also cuts the
maximum error between the arrow and the actual direction to ~12deg.

Make the OSD home arrow closer to the actual home direction
We were using the first image (up) for [0deg, 22deg] which made
some directions like [350deg, 360deg) feel pretty unnatural because
the craft was almost pointing home but the arrow pointed a bit to
the left. Maximum error between actual direction and the arrow was
22deg.

With this change, the up image is used for [349deg, 11deg], 2nd
image covers [12deg, 33deg], etc... which makes the error between
the actual direction and the arrow smaller. This also cuts the
maximum error between the arrow and the actual direction to ~12deg.

@fiam fiam added this to the 2.0 milestone Jun 18, 2018

@fiam fiam merged commit 3a87610 into development Jun 22, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fiam fiam deleted the agh_osd_home_arrow_reduce_max_error branch Jun 22, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.