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

Trainer Mode Master/Serial turns Off when reading models and settings from radio to Companion #5676

Closed
1 task done
clarke-john opened this issue Nov 18, 2024 · 5 comments · Fixed by #5684
Closed
1 task done
Assignees
Labels
bug 🪲 Something isn't working companion Related to the companion software

Comments

@clarke-john
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?

Companion

Current Behavior

When I ‘Read Models and Settings from Radio’ to Companion the setting for the ‘Trainer Mode’ changes from ‘Master/Serial’ to ‘Off’.
‘Write Models and Settings to Radio’ from Companion works fine and the Trainer Mode setting is not lost in this transfer.
I noticed this bug in February (version 2.9.4) but didn't report it.

Expected Behavior

I would expect the Trainer Mode setting on the radio to be transferred to Companion.

Steps To Reproduce

On a model in the radio, set 'Trainer Mode’ to ‘Master/Serial’.
Open Companion and connect radio to PC.
In Companion click on ‘Read Models and Settings from Radio’.
Open the model.
Check the setting for the ‘Trainer Mode’ and note that it has changed from ‘Master/Serial’ to ‘Off’.

Version

2.10.5

Transmitter

FrSky X10 / X10S (ACCST)

Operating System (OS)

Windows

OS Version

W11

Anything else?

No response

@clarke-john clarke-john added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Nov 18, 2024
@pfeerick pfeerick added companion Related to the companion software and removed triage Bug report awaiting review / sorting labels Nov 24, 2024
@pfeerick
Copy link
Member

pfeerick commented Nov 24, 2024

I am able to reproduce this on TX16S also.

  • Set AUX1 to "SBUS Trainer", to enable the "Master/Serial" trainer option
  • Configure Trainer Mode to be Master/Serial on handset
  • Companion reads as OFF, although "Master/Serial" is in the list of options

Also, "Master/SBUS Module" is missing as a trainer option in Companion, and results in a blank (not OFF) field.

@elecpower
Copy link
Collaborator

Thanks @pfeerick more work lol

@pfeerick
Copy link
Member

Yeah, I know... that todo list never gets any shorter! 😆

@pfeerick pfeerick self-assigned this Nov 24, 2024
@pfeerick
Copy link
Member

Hm... #4266 is at least part of the reason for this, as it dynamically checks if the radio has hardware capabilities (which allowed a few more radios to have SBUS/CPPM option and also checks if it is in use by something else...

@clarke-john
Copy link
Author

clarke-john commented Nov 30, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working companion Related to the companion software
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants