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

fms: FPLN VNAV predictions not taking entered block fuel into account #8639

Closed
BravoMike99 opened this issue Apr 25, 2024 · 3 comments
Closed
Labels
Bug Something isn't working FMS - Predictions Good First Issue Good for newcomers

Comments

@BravoMike99
Copy link
Contributor

BravoMike99 commented Apr 25, 2024

Aircraft Version

Development

Build info

{
    "built": "2024-04-24T17:04:51+00:00",
    "ref": "refs/heads/master",
    "sha": "c989ce225b7a6c201346dfc73609ea45eb203ff0",
    "actor": "frankkopp",
    "event_name": "manual",
    "pretty_release_name": "master:c989ce22",
    "version": "a32nx-v0.12.0-dev.c989ce2"
}

Describe the bug

FMS predictions take into account the FOB instead of the block fuel entered on the INIT FUEL PRED page before engine start. This is especially noticeable when refueling.
image
image
image

Expected behavior

The block fuel entered on the fuel pred page should be used for predictions prior to engine start. Once the first engine is started, the sensed fuel on board should be used.

Steps to reproduce

  1. Fill INIT A, INIT FUEL PRED & Perf takeoff with any data.
  2. Change the fuel on board via the EFB and observe how the EFOB at destination changes on the F-PLN page.

References (optional)

FCOM DSC-22_20-50-10-25 -> INIT FUEL PRED Page

Additional info (optional)

No response

Discord Username (optional)

bruno_pt99

@BravoMike99 BravoMike99 added the Bug Something isn't working label Apr 25, 2024
@patmack14
Copy link
Contributor

Ill get going on this one!

patmack14 added a commit to patmack14/a32nx that referenced this issue May 27, 2024
BlueberryKing added a commit that referenced this issue Sep 21, 2024
* FIx for #8639

* Update .github/CHANGELOG.md

Co-authored-by: Michael Corcoran <tracer@outlook.co.nz>

* Update A32NX_FMCMainDisplay.js

removed comments and extra conversion

* adding to make sure an engine is running in AOC

* additional gw safety

Added an additional check to check for block fuel being entered to prevent "A32NX_FM_GROSS_WEIGHT" from being Null

* updates per @BlueberryKing suggestions

Add .isFinite type safety for and isAnEngineOn()  logic instead of currentFoB()

* forgot to remove 106->108..doh!

* Fix formatting

---------

Co-authored-by: Michael Corcoran <tracer@outlook.co.nz>
Co-authored-by: BBK <22713769+BlueberryKing@users.noreply.github.com>
tracernz added a commit to tracernz/a32nx that referenced this issue Sep 28, 2024
…8664)

* FIx for flybywiresim#8639

* Update .github/CHANGELOG.md

Co-authored-by: Michael Corcoran <tracer@outlook.co.nz>

* Update A32NX_FMCMainDisplay.js

removed comments and extra conversion

* adding to make sure an engine is running in AOC

* additional gw safety

Added an additional check to check for block fuel being entered to prevent "A32NX_FM_GROSS_WEIGHT" from being Null

* updates per @BlueberryKing suggestions

Add .isFinite type safety for and isAnEngineOn()  logic instead of currentFoB()

* forgot to remove 106->108..doh!

* Fix formatting

---------

Co-authored-by: Michael Corcoran <tracer@outlook.co.nz>
Co-authored-by: BBK <22713769+BlueberryKing@users.noreply.github.com>
@CaptCrystal
Copy link

Can this issue be closed since #8664 was merged?

@BlueberryKing
Copy link
Member

Can this issue be closed since #8664 was merged?

Yes, indeed, good catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working FMS - Predictions Good First Issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants