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

Expand YMMB Procedures #385

Merged

Conversation

glennawatson
Copy link
Contributor

@glennawatson glennawatson commented Nov 24, 2024

Summary

Expand YMMB procedures to include preferred duty runways, missing inbound waypoint SHOL, and helicopter details.

Changes

Fixes:

  • Duty runway preferences are 17/35 at MB

Changes:

  • Cleaned sections up to read a bit clearer

Additions:

  • Helicopter overview
  • SHOL inbound point

@glennawatson
Copy link
Contributor Author

Images incoming

@glennawatson
Copy link
Contributor Author

glennawatson commented Nov 24, 2024

Would like some clarification what we'd like to do with helicopters

Real life MB controller indicates they don't use the aiming points due to them infringing on the runway environment but it's in the ERSA. Do we want to follow the ERSA or MB controller or something else?

Copy link
Collaborator

@mattkelly4 mattkelly4 left a comment

Choose a reason for hiding this comment

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

Looks like a heap of work has gone into this, thanks as always Glenn. Just a few discussion points from my end, mainly standardisation items or queries based on my lack of experience with the aerodrome. Feel free to reach out to discuss any points if you wish!

docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
@mattkelly4
Copy link
Collaborator

Would like some clarification what we'd like to do with helicopters

Real life MB controller indicates they don't use the aiming points due to them infringing on the runway environment but it's in the ERSA. Do we want to follow the ERSA or MB controller or something else?

Have replied via discord but for everyone else's visibility, I think we'll stick with what the ERSA says so that pilots and controllers are operating with a common source of information. If this becomes problematic, we can look at amending the procedure and including the changes in the Pilot Procedures site.

docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
docs/aerodromes/classd/Moorabbin.md Outdated Show resolved Hide resolved
@glennawatson
Copy link
Contributor Author

What's left to get this one in, doing the diagram for it today

@tylerthetiletiler tylerthetiletiler added the notify Send notification to Discord on merge label Dec 22, 2024
@glennawatson
Copy link
Contributor Author

Diagrams are in, had to fix a problem I was having with the stylesheet for the phraseology section stripping out new lines.

All feedback applied. Ready when you guys are back from the break @alphadelta332 and @mattkelly4 and anyone who can review.

Would require every example to be reformatted
@mattkelly4
Copy link
Collaborator

Diagrams are in, had to fix a problem I was having with the stylesheet for the phraseology section stripping out new lines.

All feedback applied. Ready when you guys are back from the break @alphadelta332 and @mattkelly4 and anyone who can review.

FYI, I reversed the stylesheet change. Really appreciate the fix but unfortunately, it would require every example on the site to be reformatted, so have left it the way it was previously. Adding the double space after each line forces a new line in example blocks (as with elsewhere).

@glennawatson
Copy link
Contributor Author

Ready for another review round @mattkelly4 and @alphadelta332

@mattkelly4 mattkelly4 added the DO NOT MERGE This PR is not ready to be merged. Do not merge until approval from the Publications Manager. label Dec 31, 2024
@mattkelly4 mattkelly4 changed the title Expand on YMMB, add helicopters, add missing waypoints, add duty runways Expand YMMB Procedures Jan 17, 2025
alphadelta332
alphadelta332 previously approved these changes Jan 26, 2025
Copy link
Collaborator

@alphadelta332 alphadelta332 left a comment

Choose a reason for hiding this comment

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

Thanks for your work again @glennawatson. We've made some formatting changes, tidying up and typos, things like that. We've included most of the work you've done, main thing that we've taken out is the Helicopter Arrival/Departure procedures, in order to reduce complexity and allow the freedom for the controller to handle aircraft similarly across multiple towers, given they're rated to do all towers in the country. It poses a low risk from an operational standpoint, since no separation standard needs to be applied to VFR choppers, the ATC can make their own determination for the passing of traffic information based on the arrival/departure procedure they elect to use.

Thanks for helping out

@alphadelta332 alphadelta332 requested review from tylerthetiletiler and a team January 26, 2025 03:14
@alphadelta332 alphadelta332 removed the DO NOT MERGE This PR is not ready to be merged. Do not merge until approval from the Publications Manager. label Jan 26, 2025
mattkelly4
mattkelly4 previously approved these changes Jan 26, 2025
@glennawatson
Copy link
Contributor Author

Given the mbhelidep.png file is no longer used, you may want to delete it.

@glennawatson
Copy link
Contributor Author

I'm actually ok with the Dep going, since what was in the ERSA and what they do in real life didn't match. So there was some ambiguity there anyway.

no longer used in the PR based on the edits
@glennawatson
Copy link
Contributor Author

@tylerthetiletiler @mattkelly4 @alphadelta332

Need approval again, removed the unused image file.

@mattkelly4 mattkelly4 merged commit 1e97dfd into vatpac-technology:main Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notify Send notification to Discord on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants