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

Wrong Cabin temperature on Cruise SD #8213

Closed
twasono opened this issue Sep 13, 2023 · 3 comments · Fixed by #8243
Closed

Wrong Cabin temperature on Cruise SD #8213

twasono opened this issue Sep 13, 2023 · 3 comments · Fixed by #8243
Labels
Bug Something isn't working Good First Issue Good for newcomers

Comments

@twasono
Copy link

twasono commented Sep 13, 2023

Aircraft Version

Development

Build info

{
    "built": "2023-09-04T02:30:14+00:00",
    "ref": "refs/heads/master",
    "sha": "12c8e357bccc95d58b6fab4d2e2051ca6b9940fb",
    "actor": "Benjozork",
    "event_name": "manual",
    "pretty_release_name": "master:12c8e357",
    "version": "v0.11.0-dev.12c8e35"
}

Describe the bug

The fwd and aft temperatures are using the LVAR for cockpit temperature.

I think it's a regression caused by this PR: #8118

I already put a comment on that PR.

Expected behavior

The fwd and aft temperatures should use their own LVAR to get the right temp

Steps to reproduce

Fly as usual and try to change the cockpit temp.

You will see that all temps (including fwd and aft) will be displayed as cockpit temp

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

No response

@twasono twasono added the Bug Something isn't working label Sep 13, 2023
@tracernz
Copy link
Member

Good catch 👍.

@BlueberryKing BlueberryKing added the Good First Issue Good for newcomers label Sep 29, 2023
@cptnuss-ops
Copy link
Contributor

Do I have to assign the issue to myself prior to fixing? Or this done by creating a PR for this issue?

@Saschl
Copy link
Contributor

Saschl commented Oct 5, 2023

Hi, it’s best you create a draft PR already which mentions this issue you want to fix.

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

Successfully merging a pull request may close this issue.

5 participants