EdgeTX / edgetx

EdgeTX is the cutting edge open source firmware for your R/C radio
https://edgetx.org
GNU General Public License v2.0
1.62k stars 340 forks source link

Companion - GVs missing from Inputs source list #3086

Closed pfeerick closed 1 year ago

pfeerick commented 1 year ago

Is there an existing issue for this problem?

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. https://github.com/EdgeTX/edgetx/issues/3086#issuecomment-1402965556

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

elecpower commented 1 year ago

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

Questions:

elecpower commented 1 year ago

Using the X9D+ libsim I can confirm that:

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