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

docs(bg): update sample flight in preparing-mcdu.md #939

Merged
merged 14 commits into from
Feb 17, 2024

Conversation

Valastiri
Copy link
Member

@Valastiri Valastiri commented Feb 5, 2024

fixes #938

Summary

As indicated through our #docs channel on Discord the routing on the beginners guide is no longer applicable to the current AIRAC 2401.

This PR updates that.

Current Changes

  • Changed our sample flight to the Germany (EDDM/EDDF)
  • Updated the PDF
  • Updated the screenshots from PDF
  • Completed initial text changes (requires final review)

To Do

  • Requires new in-sim MCDU Photos
  • Confirm that the text matches the imagery.

Location

  • docs/pilots-corner/beginner-guide/preparing-mcdu.md
  • docs/pilots-corner/assets/beginner-guide/mcdu/[...]

Discord username (if different from GitHub):
valastiri

Copy link

vercel bot commented Feb 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 14, 2024 2:01pm

@Valastiri Valastiri marked this pull request as ready for review February 10, 2024 17:48
@github-actions github-actions bot added the Review Required PR Check Label label Feb 10, 2024
Copy link
Member

@frankkopp frankkopp left a comment

Choose a reason for hiding this comment

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

Initial review - I'll work through it with the aircraft tomorrow.

@@ -302,9 +305,11 @@ Verify the flight plan by using the vertical slew keys to scroll through it.

The flight plan might contain so-called discontinuities. These are breaks in the flight plan and often separate two flight plan sections, like the SID and first in-route waypoint or the STAR and the APPR. They are also often inserted when the flight plan is modified.

Sometimes discontinuities are also part of a procedure to indicate that manual input is required (mostly clearance by ATC). The preceding legs are called MANUAL legs.
Sometimes discontinuities are also part of a procedure to indicate that manual input is required (mostly clearance by ATC). The preceding legs are called MANUAL legs.In this flight plan we have a discontinuity between the STAR and approach procedure as shown below.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Sometimes discontinuities are also part of a procedure to indicate that manual input is required (mostly clearance by ATC). The preceding legs are called MANUAL legs.In this flight plan we have a discontinuity between the STAR and approach procedure as shown below.
Sometimes discontinuities are also part of a procedure to indicate that manual input is required (mostly clearance by
ATC). The preceding legs are called MANUAL legs. These cannot and should not be deleted. See below link on how to
handle these.
In this flight plan we have a discontinuity between the STAR and approach procedure as shown below.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm on GitHub but I might have forgotten to include it (and also forgot the title) -- but I meant to show a photo here of a MANUAL and F-PLN Disco just before we offload the user to a new page.

Minus one suggestion. Comment left.

Co-authored-by: Frank Kopp <[email protected]>
@@ -212,29 +212,31 @@ Our route for this flight can be found on the 2nd page of the OFP

To program the Standard Instrument Departure (SID):

* Press LSK1L or EGFF (the departure airport)
* Press LSK1L or EDDM (the departure airport)
* Select `DEPARTURE` shown next to LSK1L
* Select the runway we are departing from. In this case `30` using LSK3L
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Select the runway we are departing from. In this case `30` using LSK3L
* Select the runway we are departing from. In this case `08L` using LSK3L

The LSK should also be corrected, didn't see a screenshot for it.

Copy link
Collaborator

@Sleinmaster Sleinmaster left a comment

Choose a reason for hiding this comment

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

One minor suggestion, otherwise good to go.

@Valastiri
Copy link
Member Author

I missed the approvals on this apologies. Thanks for the assist :)

@github-actions github-actions bot removed the Review Required PR Check Label label Feb 17, 2024
@github-actions github-actions bot merged commit 0d85380 into flybywiresim:primary Feb 17, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update beginner guide Setup MCDU route
3 participants