-
Notifications
You must be signed in to change notification settings - Fork 6
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
Change all stands to polygons #406
Conversation
COORD:N063.27.16.766:E010.55.13.428 | ||
COORD:N063.27.19.119:E010.55.13.562 | ||
COORD:N063.27.19.124:E010.55.10.403 | ||
COORD:N063.27.16.798:E010.55.10.308 | ||
CODE:C | ||
ADEP:EN |
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.
Are these stands domestic only? They dont have a connection to the terminal?
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.
Need to check that. Might be flex stands, but i have never seen them actually used.
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.
You are referring to stand 38 and 39, right?
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.
Correct
STAND:ENGM:319 | ||
COORD:N060.12.03.017:E011.04.36.683 | ||
COORD:N060.12.02.105:E011.04.36.849 | ||
COORD:N060.12.01.909:E011.04.31.987 | ||
COORD:N060.12.02.832:E011.04.31.877 | ||
COORD:N060.12.03.017:E011.04.36.683 |
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.
Need to test the position of this polygon somewhat. Either it's a bit off or the label should be moved.
This PR is now ready for review and merging. |
So these stands that are spaced out from each other I would probably make it non overlapping as done with 171 and 171L. Since if someone park on 208R, 203L, 201L even with a B738, it would most likely block the other stands regardless. If they spawn on 208, it might misdetect them on 208R and therefore also blocking 207. If this made sense? It is not possible to do this everywhere, for example at ENVA with stand 40 and 40L |
COORD:N060.13.10.278:E011.06.14.135 | ||
COORD:N060.13.08.843:E011.06.11.886 | ||
COORD:N060.13.09.540:E011.06.10.148 | ||
COORD:N060.13.10.963:E011.06.12.343 | ||
USE:M | ||
PRIORITY:-3 |
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.
Would remove autoassignment here - would be very weird to park someone here based on automatic stand assignment.
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.
Changed to manual in 8fb4d46.
Not sure if pilot just overshoot or not.. But could probably be adjusted a bit. |
See if these changes from 672fbc8 works better. |
Looks quite abit better, can same be done with 188/189? |
Fixed in a0ea962 now. |
They are quite close together, but i've made an adjustment in e266fdc now. |
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.
Nice work, lots of changes but I think it should be all good now 👍🏻
ENBR
ENVA
ENTO
ENZV
ENGM
Closes #383