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

Taranis X-Lite trim switch does not match stick mode #5332

Closed
1 task done
chrisu365 opened this issue Jul 27, 2024 · 4 comments · Fixed by #5580
Closed
1 task done

Taranis X-Lite trim switch does not match stick mode #5332

chrisu365 opened this issue Jul 27, 2024 · 4 comments · Fixed by #5580
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting

Comments

@chrisu365
Copy link

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

Have a look to opentx/opentx#8260
Setting mode 2 stick configuration the trim switch works for left stick without shift and for right stick with shift key.

Expected Behavior

Trim should follow the stick mode setting so that the "unshifted" trim keys always trim aileron and elevator as these are the most critical trims

Steps To Reproduce

  1. set transmitter stick mode to 'mode 2' (aileron on right stick)
  2. try left/right trim --> it takes effect on left stick
  3. try up/down trim --> it takes effect on left stick
  4. set transmitter stick mode to 'mode 4' (aileron on left stick)
  5. try left/right trim --> it takes effect on left stick
    6.try up/down trim --> it takes effect on left stick

Version

2.10.2

Transmitter

FrSky X-Lite / S / Pro

Operating System (OS)

Windows

OS Version

Windows

Anything else?

No response

@chrisu365 chrisu365 added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Jul 27, 2024
@chrisu365
Copy link
Author

chrisu365 commented Sep 25, 2024

As i am using stickmode 2 only, i modified KEY_SHIFT to compile my own version using the following code:
https://github.com/EdgeTX/edgetx/tree/main/radio/src/boards/generic_stm32/inputs.cpp
org:
#if defined(PCBXLITE)
if (_read_keys() & (1 << KEY_SHIFT))

new:
#if defined(PCBXLITE)
if ((_read_keys() & (1 << KEY_SHIFT))==0)

@markran
Copy link

markran commented Oct 1, 2024

I just posted on RCG asking about this and was referred here. Happy to see it was already reported. Hopefully, this regression will be addressed shortly!

@pfeerick
Copy link
Member

pfeerick commented Oct 4, 2024

@markran As an X-Lite owner (presumably), would you be able try PR #5580 PR, which hopefully fixes this on hardware 🤞

To access the automatic builds for a Pull Request (PR) you are interesting in trying out, first open up page for the PR in question. Under the title for the pull request, you should see a series of tabs (Comments, Commits, Checks, Files Changed). Click on the Checks tab.

If you are after the radio firmware, click on the link on the left side mentioning Firmware.

Then, under the 'Artifacts' heading near the bottom of the page will be a link to the zip file containing either the firmware or companion package.

@chrisu365
Copy link
Author

I installed the artefact from: https://github.com/EdgeTX/edgetx/actions/runs/11134881058/job/30944247037?pr=5580
It works on my real X-Lite as expected
Thanks a lot !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants