search

zlsa / atc

https://openscope.co/
342 stars 107 forks source link

Added delete command #605

Closed glott closed 8 years ago

glott commented 8 years ago

Added delete command. Can be used to remove aircraft from the scope. Syntax: [Select aircraft] delete Synonyms: del, rem

tedrek commented 8 years ago

I'm curious what is the intended use case for this? It is certainly not very realistic :wink:

erikquinn commented 8 years ago

It's present in all the major simulators (NexSim, Adacel, UFA, etc), and I can attest there are times I've wanted it... @tedrek it's definitely not realistic, but it's important from the training perspective, as it allows you to focus on a particular task (say box pattern vectoring), and if you mess up, you have the option to kill a plane or two and continue with the drill. Useful for developing skills, and personally i would rather have the option to use or not use it based on whatever purposes I'm playing the game that day.

tedrek commented 8 years ago

@erikquinn, thanks for the insight. I'll merge once my comment on the diff is resolved.

glott commented 8 years ago

I'll fix it when I have a chance. What do you think would be the best/most efficient way to give the option of deleting an aircraft when they are not in the center airspace? @erikquinn @tedrek

tedrek commented 8 years ago

@JHG0, the best solution is probably to wait for #601 to be merged and then implement it as a system level command which will delete regardless of being in the airspace.

glott commented 8 years ago

Closed - new pull request will be made after merge with #601. @tedrek