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

Waypoint routes #22

Closed
zlsa opened this issue Dec 15, 2014 · 3 comments
Closed

Waypoint routes #22

zlsa opened this issue Dec 15, 2014 · 3 comments
Labels
enhancement

Comments

@zlsa
Copy link
Owner

zlsa commented Dec 15, 2014

Multiple fixes in order.
@zlsa zlsa closed this as completed in 0c11234 Dec 15, 2014
@niklasfi
Copy link

How would you feel about a complete revamp of navigation requests? I have the following semantics in mind:

  • plane f abofa a 2 f kaito s 100 f goody l 36r would mean: fly to abofa, uppon arrival at abofa change altitude to 2000ft and fix kaito. Uppon arrival at kaito decelerate to 100 and fix goody. Uppon arrival at goody initiate landing on runway 36R.
  • plane a 8 fix divec h 0 a 4 would mean: immediately change altitude to 8000ft and fix divec. Uppon arrival at divec change heading to 0 bearing and change altitude to 4000ft.

As you can see, the fix command creates a barrier in the queue. Commands following the fix command are executed uppon arrival at the fix.

I realise this would take a major rewrite of the navigation system. This is why I would like to know wether you are interested in a patch, that implements the described behaviour.

@zlsa
Copy link
Owner Author

zlsa commented Dec 16, 2014

Is that syntax used in real life? If it is, I'd gladly accept a patch.

@niklasfi
Copy link

I can't really comment on that since I have never sat in front of an atc terminal but I feel this user interface is most intuitive.

Will get to work later today.

@niklasfi niklasfi mentioned this issue Dec 18, 2014
Merged
erikquinn pushed a commit that referenced this issue Nov 12, 2016
@n8rzz
feature/ATC-12 - adds http-server and start script to package.json …
6b62d16 
…that fires up a local server (#22)
erikquinn pushed a commit that referenced this issue Nov 21, 2016
@n8rzz
- bugfix/ATC-4 - updates .ready() method to use a localStorage icao o…
405e078 
…n load if one exists (#14)

- feature/ATC-12 - adds http-server and `start` script to package.json that fires up a local server (#22)
- Adds comments to ArrivalBase for future refactor work
- Simplifies airline_get method and abstracts addAirline() into its own method.
- Abstracts parse logic into helper methods. Updates instances of for/in loops to use forEach instead.
- Changes .createChildren() method names to ._init()
- Adds doc blocks to new AirportModel methods.
- Abstracts if logic out of AirportModel.parse()
- Updates getFix to getFixPosition in AirportModel
- Updates references
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@niklasfi @zlsa