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

Negative RC threshold not working in v1.13.0 #19809

Closed
Katawann opened this issue Jun 15, 2022 · 6 comments · Fixed by #21796
Closed

Negative RC threshold not working in v1.13.0 #19809

Katawann opened this issue Jun 15, 2022 · 6 comments · Fixed by #21796

Comments

@Katawann
Copy link

Katawann commented Jun 15, 2022

Describe the bug

I would like to trigger the RTL mode when the switch is at its low value instead of high. In v1.10.1 I could use the parameter RC_RETURN_TH with a negative value to obtain this behavior. The same parameter in v1.13.0-rc1 has no effect (the RTL mode is not triggered at all when you set a negative value). The workaround is to change the RCX_REV of the corresponding channel but it is not the best solution as this value is cleared when you redo a radio calibration

To Reproduce

Steps to reproduce the behavior:

  1. Set the Return mode to a channel
  2. Change the RC_RETURN_TH to a negative value >= -1
  3. The RTL is not triggered anymore

Expected behavior

Threshold when lower than the value instead of higher

I guess the issue is the same with all RC mode. Thank you for your help !

@brunweil
Copy link

not sure but it's looks like: I get something similar with Flap on chanel 6:
in V1.12.3 minFlap with 1000us, half flaps at 1500us, full flap at 2000us
in v1.13.0 minFlap with 1000us, always minFlap at 1500us & full flap at 2000us

@junwoo091400 junwoo091400 self-assigned this Jul 5, 2022
@Katawann
Copy link
Author

Katawann commented Feb 1, 2023

Hi @junwoo091400,

The problem is still existing on the v1.13.2. Do you have any update regarding this issue or a fix in main branch that I missed ?

Thank you for your help !

@Katawann
Copy link
Author

Katawann commented Jun 8, 2023

Temporally fixed by setting RC6_REV (used in my case for the RTH) to 1. Hopefully it will be fixed in 1.14

This need to be done after calibration of the RC

@Katawann Katawann closed this as completed Jun 8, 2023
@junwoo091400
Copy link
Contributor

The problem is still existing on the v1.13.2. Do you have any update regarding this issue or a fix in main branch that I missed ?

Hi @Katawann I missed this! I have added this to the v1.14 project board to keep track of it 🙌, thanks for the report and reminder!

@Katawann
Copy link
Author

Hi @junwoo091400,

I reopen the issue as I just tried on v1.14 and the issue is still there

@junwoo091400
Copy link
Contributor

I have created a Fix PR for this linked above @Katawann 🙏. Thank you again for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment