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

fix(ND): fix the color and orientation of course reversal arrow #8185

Merged
merged 4 commits into from
Sep 2, 2023

Conversation

Maximilian-Reuter
Copy link
Member

@Maximilian-Reuter Maximilian-Reuter commented Aug 28, 2023

Fixes #[issue_no]

Summary of Changes

This fixes the color of the course reversal arrow so it no longer becomes magenta or amber depending on the vnav constraint prediction at the waypoint

It also fixes the orientation of the arrow since it used to follow the inbound course but no longer does in the current NDv2 implementation.

Screenshots (if necessary)

Before:
image

After:
image

References

Additional context

Discord username (if different from GitHub): _chaoz_

Testing instructions

You can force the course reversal arrow by inserting a hold at a waypoint.
You need to choose a waypoint with a vnav constraint, or add one yourself.

The prediction will only show on waypoints that are a few waypoints ahead from your current waypoint.

For the orientation, just play with different courses for the hold and the arrow should change accordingly

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, click on the bottom PR tab
  4. Click on the A32NX download link at the bottom of the page

@Maximilian-Reuter Maximilian-Reuter changed the title fix(ND): fix the color of course reversal arrow fix(ND): fix the color and orientation of course reversal arrow Aug 29, 2023
Copy link
Contributor

@Saschl Saschl left a comment

Choose a reason for hiding this comment

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

LGTM

@alepouna
Copy link
Member

alepouna commented Sep 2, 2023

QA Report

Discord: alepouna
Object of testing: #8185
Tier of Testing : 1
Date : 02/09/2023

Testing Process:

  • Inserting a hold at a waypoint (that has a constraint)

Testing Results:
Passed

@2hwk 2hwk merged commit 93ac522 into flybywiresim:master Sep 2, 2023
6 checks passed
@2hwk 2hwk added this to the v0.11.0 milestone Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✔️ Done
Development

Successfully merging this pull request may close these issues.

5 participants