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
base: develop
Are you sure you want to change the base?
Bugfix/2075 #2086
Conversation
Your Render PR Server URL is https://openscope-develop-pr-2086.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-cfvc45seoogsgno1mo20. |
assets/airports/kdab.json
Outdated
@@ -561,7 +561,7 @@ | |||
"destination": "kdab", | |||
"category": "arrival", | |||
"route": "FOBIN.Q93.MALET..OMN", | |||
"altitude": 23000, | |||
"altitude": 6000, |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
There was a problem hiding this 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!
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)