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

feat(hyd): changed gear lever mechanism / added gear door drag #7390

Merged
merged 40 commits into from
Dec 29, 2022

Conversation

crocket63
Copy link
Contributor

@crocket63 crocket63 commented Aug 1, 2022

Fixes #[issue_no]

Summary of Changes

Default gear lever mechanism using GEAR HANDLE POSITION simvar causes bugs with ground collision detection (landing gear down with gear lever up causes wheels going through the ground).

A custom lvar is added to handle the lever position only, and simvar GEAR HANDLE POSITION is built by system wasm depending on actual gear extension position

Added features:

  • Gear doors opened cause drag
  • Gear lever animation shows it moving a bit when it's locked down

Screenshots (if necessary)

References

Additional context

Discord username (if different from GitHub):

Testing instructions

Check everything that can be affected by a massive gear system hack.

Non regression check non exhaustive list:

  • Gear deployed still adds drag to the plane
  • Systems using gear state and lever state are working properly (landing gear light...)
  • Sound of gear handle is still working
  • Check of gear inputs using buttons / cockpit clics / mouse drag ...
    ...

New features:

  • Gear lever up. Gravity extension to get gear down with gear lever still up. Check plane lands normally and brakes correctly.
  • Doors opened only (using gravity gear extension 2 turns only) adds some drag to the plane
  • Collision detection works if you land with not all gears downlocked (use failures)
  • Trying to pull gear lever up on ground shows the lever moving a bit up but refuses to go full up
  • Lever locked and refusing to go up should produce a new sound

Check that lever can be manipulated by mousewheel and drag in lock and legacy interaction modes.

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

@crocket63 crocket63 changed the title fix(hyd): changed gear lever mechanism fix(hyd): changed gear lever mechanism / added gear door drag Aug 3, 2022
@crocket63 crocket63 changed the title fix(hyd): changed gear lever mechanism / added gear door drag feat(hyd): changed gear lever mechanism / added gear door drag Aug 3, 2022
Saschl added a commit to Saschl/a32nx that referenced this pull request Dec 2, 2022
Saschl added a commit to Saschl/a32nx that referenced this pull request Dec 8, 2022
@Saschl Saschl added the Exp Available on experimental branch (for testing) label Dec 12, 2022
Saschl added a commit that referenced this pull request Dec 12, 2022
Saschl added a commit that referenced this pull request Dec 22, 2022
beheh pushed a commit that referenced this pull request Dec 22, 2022
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.

LGTM

@crocket63 crocket63 merged commit 248fe73 into flybywiresim:master Dec 29, 2022
@crocket63 crocket63 deleted the feat_custom_gear_lever_var branch January 13, 2023 13:15
frankkopp pushed a commit that referenced this pull request Jan 19, 2023
frankkopp pushed a commit that referenced this pull request Jan 22, 2023
frankkopp pushed a commit that referenced this pull request Jan 23, 2023
frankkopp pushed a commit that referenced this pull request Jan 24, 2023
frankkopp added a commit that referenced this pull request Jan 27, 2023
feat(hyd): changed gear lever mechanism / added gear door drag (#7390)
(Rebase Fix: cherry-picked from commit 248fe73)
Rebase fixes
Update to README files (fixes)
Update to README files
Adding a workflow for the branch to upload to CDN
Fix a gitignore issue
Fixing to build workflow for fragmenter
Fixing A380 wasm build
Update to eslint config and fixes for linter errors
Updated script permissions for github actions
Updated dev-env links to new environment
Rebase fixes
frankkopp added a commit that referenced this pull request Jan 28, 2023
feat(hyd): changed gear lever mechanism / added gear door drag (#7390)
(Rebase Fix: cherry-picked from commit 248fe73)
Rebase fixes
Update to README files (fixes)
Update to README files
Adding a workflow for the branch to upload to CDN
Fix a gitignore issue
Fixing to build workflow for fragmenter
Fixing A380 wasm build
Update to eslint config and fixes for linter errors
Updated script permissions for github actions
Updated dev-env links to new environment
Rebase fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Exp Available on experimental branch (for testing) QA Tier 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants