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

Companion - GVs missing from Inputs source list #3086

Closed
1 task done
pfeerick opened this issue Jan 23, 2023 · 2 comments · Fixed by #3102 or #3097
Closed
1 task done

Companion - GVs missing from Inputs source list #3086

pfeerick opened this issue Jan 23, 2023 · 2 comments · Fixed by #3102 or #3097
Assignees
Labels
B&W Related generally to black and white LCD radios bug 🪲 Something isn't working color Related generally to color LCD radios companion Related to the companion software

Comments

@pfeerick
Copy link
Member

pfeerick commented Jan 23, 2023

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

I noticed when looking at one version of a FM setup I use that GVs are not being shown on the GVs source list for colorlcd. It seems this capability is only valid for colorlcd - B&W don't seem to have this, which is possibly why this is missing. This means if I were to read/write the model the setting would most likely be trashed.

cf. #3086 (comment)

Expected Behavior

That GVs appear in the Inputs source list (between Channels and Telemetry entires), like on the transmitter/simulator.

Steps To Reproduce

  1. Configure a Input on TX16S with GV as the source
  2. Read that model with Companion
  3. See that the Inputs source field is blank in Companion

Version

2.8.0

Transmitter

Radiomaster TX16S / TX16SMK2

Anything else?

Present on both release and main build of firmware/Companion.

Attached is the model file I noticed this on.
Skysurfer-ELRS.zip

@pfeerick pfeerick added bug 🪲 Something isn't working color Related generally to color LCD radios companion Related to the companion software labels Jan 23, 2023
@elecpower
Copy link
Collaborator

elecpower commented Jan 23, 2023

From a quick check GV is not an available Input source for colorlcd in OTX so appears added in ETX.

Questions:

  • are there any differences in the mixer engine code between radios types?
  • if not, why are GVs not available on B&W edit ui?
  • GV is not available as a source in Mixes edit ui for colorlcd, why not if available for Inputs?
  • is GV source for a Mix handled in mixer engine code?
  • etc

@elecpower
Copy link
Collaborator

elecpower commented Jan 25, 2023

Using the X9D+ libsim I can confirm that:

  • it does load the yml input GV source
  • the ui does not let you edit an input source to a GV but does display
  • the mixer engine does not discriminate between B&W and color.

Based on these results there needs to be:

screenshot_x9d+_23-01-25_12-33-00
screenshot_x9d+_23-01-25_12-32-20
screenshot_x9d+_23-01-25_12-40-58
screenshot_x9d+_23-01-25_12-32-35

@pfeerick pfeerick changed the title Companion - colorlcd - GVs missing from Inputs source list Companion - GVs missing from Inputs source list Jan 25, 2023
@pfeerick pfeerick added the B&W Related generally to black and white LCD radios label Jan 25, 2023
@elecpower elecpower linked a pull request Jan 25, 2023 that will close this issue
@elecpower elecpower self-assigned this Jan 25, 2023
@elecpower elecpower linked a pull request Jan 25, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B&W Related generally to black and white LCD radios bug 🪲 Something isn't working color Related generally to color LCD radios companion Related to the companion software
Projects
None yet
2 participants