edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Enabling both internal and external ELRS should be usable or impossible

Open wimalopaan opened this issue 9 months ago • 4 comments

Is there an existing issue for this problem?

  • [X] I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

Oviously ETX is not designed to use two ELRS modules anabled at the same time. Telemetry -> "Start search sensors" completely hangs the radio (need to disconnect the battery).

Expected Behavior

It is possible to physically have two ELRS modules in one radio: internal and external.

Either:

a) If one of the modules is enabled, keep the other disabled

b) allow that both are enabled, and then use them in the expected behaviour. This includes telemetry (EdgeTx then has to process two telemetry streams) and the elrsV3.lua, which needs to choose between the two.

Steps To Reproduce

  1. Enable both modules for a specific model
  2. In "Telemetry" start "Search sensors"
  3. the radio freezes.

Version

Other (Please specify below)

Transmitter

FrSky X12

Operating System (OS)

Linux

OS Version

Arch Linux

Anything else?

No response

wimalopaan avatar Apr 28 '24 12:04 wimalopaan

Didn't know it would make it crash, but unexpected/indeterminate behaviour was expected, but is probably only documented deep in some telemetry related issues where telemetry needs to be overhauled to allow for seperate streams for multiple modules. Disabling both is not really an option even know, as you can use both (i.e. for dual independent Rx , or dual frequency), just can't manage telemetry or Lua with both connected at the same time. Similar to using multiple rxs with a single tx for extra channels, maybe disabling telemetry on second tx/rx pipelinemay help here.

pfeerick avatar Apr 29 '24 00:04 pfeerick

Updating the internal and the external ELRS module to 3.3.4-RCx fixed the freeze problem.

wimalopaan avatar Apr 29 '24 14:04 wimalopaan

It shouldn't be missing much, as most things are already separate.

raphaelcoeffic avatar Apr 30 '24 04:04 raphaelcoeffic

Well, I don't know, what the problems with the actual implementation is, so I can't imagine to nearby solution. I do not even know for sure if this is also true for the MPM.

wimalopaan avatar Apr 30 '24 10:04 wimalopaan