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

EDGETX 2.8.1 on X7 Access: event of the key that switches from any main to the telemetry view is propagated to the telemetry script #3253

Closed
1 task done
bug400 opened this issue Feb 24, 2023 · 1 comment
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting

Comments

@bug400
Copy link
Contributor

bug400 commented Feb 24, 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?

Other (Please specify below)

Current Behavior

If I open a view of a telemetry script from any main view, the script catches a EVT_VIRTUAL_PREV_PAGE key event which originates from the long press of the PAGE key which switched to the telemetry script view. It happens on radio and on simulator.

Output of my test script is:
...
1,24s: YAML radio settings writer
1,24s: not found
1,24s: f_open(/tmp/etx-TQEzEm/RADIO/radio_new.yml, a) = 0x7f5e54000d10 (FIL 0x7f5e74980ae0)
1,24s: f_close(0x7f5e54000d10) (FIL:0x7f5e74980ae0)
1,24s: generalSettings written with checksum 42572
1,24s: f_unlink(/tmp/etx-TQEzEm/RADIO/radio.yml) = OK
1,24s: f_rename(/tmp/etx-TQEzEm/RADIO/radio_new.yml, /tmp/etx-TQEzEm/RADIO/radio.yml) = OK
5,95s: chainMenu(0, 0x7f5e76aa78af)
Other event 99
VIRTUAL PREVIOUS PAGE
Other event 4097

Expected Behavior

The event of the key which is used to switch from any main to the telemetry view is not propagated to the telemetry script.

Steps To Reproduce

Configure model with the attached telemetry script. Switch to telemetry view. Watch output.

Version

Other (Please specify below)

Transmitter

FrSky Q X7 / Q X7S (ACCESS)

Anything else?

EDGETX 2.8.,1, built with cmake -DPCB=X7 -DPCBREV=ACCESS DAUTOUPDATE=YES -DPXX1=YES -DDEFAULT_MODE=2 -DMULTIMODULE=NO -DCROSSFIRE=YES -DAFHDS3=NO -DGVARS=YES -DLUA=NO_MODEL_SCRIPTS -DTRANSLATIONS=DE -DFRSKY_RELEASE=YES -DDEFAULT_TEMPLATE_SETUP=17 -DHELI=NO -DPPM_UNIT=US -DLUA_MIXER=YES .

Test script:
test.zip

@bug400 bug400 added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Feb 24, 2023
@bug400
Copy link
Contributor Author

bug400 commented Nov 13, 2023

Fixed in 2.9.2. Testted successfully on QX7 ACCESS radio.

@bug400 bug400 closed this as completed Nov 13, 2023
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

No branches or pull requests

1 participant