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.

Sign up for GitHub

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

Jump to bottom

Add rough support for ILS approaches #12

Merged
merged 10 commits into from Sep 9, 2022
Merged

Add rough support for ILS approaches #12

merged 10 commits into from Sep 9, 2022

Conversation

dhleong
Copy link
Owner

@dhleong dhleong commented Sep 9, 2022

This is all very rough and mostly placeholder-y but lays a lot of groundwork.

  • Add hacky initial approach steering implementation
  • Begin work to detect whether an aircraft is intercepting the localizer
  • Include elevation in runway-coords
  • Attempt to detect glide slope
  • Attempt to update altitude when on glide slope to approach runway
  • Implement "cancel approach clearance" command
  • Fix runway parsing
  • Add rough support for handling aircraft landing

@dhleong
Add hacky initial approach steering implementation
5b231ed 
Currently we just steer directly toward the airport when we've
intercepted the localizer, which *may* look okay but is certainly hacky.
Still need to implement localizer math, altitude adjustments, and then
finally detect landing
@dhleong
Begin work to detect whether an aircraft is intercepting the localizer
92d4af1
@dhleong
Include elevation in runway-coords
71a14b0
@dhleong
Attempt to detect glide slope
9e9f41c
@dhleong
Attempt to update altitude when on glide slope to approach runway
65a6a0b 
We probably want to do something smarter here, like compute the
*expected altitude* for the glideslope, and apply that instead.
@dhleong
Implement "cancel approach clearance" command
2bb7afc
@dhleong
Fix runway parsing
3cb9182
@dhleong
Add *rough* support for handling aircraft landing
e1cbbda
@dhleong
Fix cancel-approach grammar to avoid false positives if unparseable
ef658e6
@dhleong
Fix local linting
0d1d0fc 
CI seems fine with this but it's struggling locally
@dhleong dhleong merged commit 31ae96d into main Sep 9, 2022
@dhleong dhleong deleted the approach-intercept branch September 9, 2022 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@dhleong