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

Bugfix/2075 #2086

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Bugfix/2075 #2086

wants to merge 4 commits into from

Conversation

svalencia014
Copy link

Resolves #2075

The purpose of this pull request is to fix southern arrivals which previously spawned at 11,000, making the controller provide vectors for the descent.
This changes those aircraft to spawn at 6,000 for the jets, and 5,000 for the props (for separation), per the vZJX DAB SOP ("F11 TRACON will assign
arrival aircraft to cross the TRACON border at no higher than 6,000").
higher than 6,000’."

(closed #2076 due to branch being renamed to follow proper naming)

@svalencia014 svalencia014 added the BUGFIX Defect of the simulator, airport, or aircraft label Mar 1, 2023
@svalencia014 svalencia014 requested a review from a team March 1, 2023 03:10
@svalencia014 svalencia014 self-assigned this Mar 1, 2023
@render
Copy link

render bot commented Mar 1, 2023

documentation/airport-guides/kdab.md Show resolved Hide resolved
documentation/airport-guides/kdab.md Outdated Show resolved Hide resolved
@@ -561,7 +561,7 @@
"destination": "kdab",
"category": "arrival",
"route": "FOBIN.Q93.MALET..OMN",
"altitude": 23000,
"altitude": 6000,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it'd be better to make use of the commands key to ensure aircraft enter the TRACON at 6000ft, rather than forcing them to spawn at 6000ft. (See https://github.com/openscope/openscope/blob/develop/documentation/spawnPatternReadme.md)

It may still be necessary to decrease the spawn altitude, but aircraft should ideally spawn at a higher level and then descend to the desired level

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing my change, I still have a plane spawning at 11,000 when I set altitude back to 23,000 and use the cross MALET A060 command in the file.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try setting the spawn altitude to something like 18,000. If that still doesn't work then I'm happy setting the spawn height to 6k.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The first plane that spawns on the FOBIN.Q93.MALET..OMN route still spawns at 11,000. All planes past the initial one on that route spawn normally and descend as instructed.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, must be something to do with the first spawn logic. We can keep it at 6000 then

Copy link
Member

@Fechulo Fechulo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reads much better now, good work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUGFIX Defect of the simulator, airport, or aircraft
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix South Arrivals at KDAB
2 participants