-
-
Notifications
You must be signed in to change notification settings - Fork 356
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
RSSI Telemety Source #2552
Comments
Do you mean when you reset/re-discover the sensors? If yes, this is the way it is. We have plans to change that, but this current limitation has been there since ever. |
Yes - Oh, okay, thanks. I thought it could be a good addition to future releases to have that selection option implemented if it is not too much effort. |
Sure, I have that on my long-term TODO list, but it's going to be tough... All the sensors are addressed by IDs which are based on discovery order.... |
Or is this issue about selecting a specific sensor as the source for "RSSI Warnings"? |
Yes, it is - since the RSSI() source seems to change from time to time while fidling with the order and names in Companion and re-discovery - although it always has the same ID. Perhaps we are talking about an issue rather than feature request. It is no big deal though, just wanted to mention it and see if there's a change on the road ahead. Once I have all things done in my telemetry, it does not get lost at all. Is there anything I can do? |
This is a valid report. There is same ticket. Missing feature from opentx ;) |
What do you mean with "it changes" then? I'm really lost. It may be instructive to see the model file before and after the change. |
I mean when I change anything in telemetry (not in the RSSI value), or with cabelling then the RSSI Source switches to NULL and that triggers events in the alarming. I had this now two times and only recognize this on the field. Does that make it clear? |
pfeerick edit: for some reason the link didn't work for me, this seems to be the discussion in question According to this discussion the option that is The question is if this setting is needed at all. Concurrently operating two FrSky module is a rare corner case. All other scenarios with one external or internal module operating don't need any rssi source settings as the default handles RSSI values associated with the module correctly. If a user really wants to concurrently operate two FrSky modules setting default will still take both modules RSSI values in account. The lower RSSI value of both modules will be used to generate low/critical warnings. This is the same behavior as having any two modules operating concurrently in ETX. So there is really no need to fiddle with rssi source. But why do some users change rssi source? Most likely because it's confusing and the behavior is not documented at all. There is ways to solve this:
3a: As an extension to 3. the rssi source settings option should also be deleted from b&w radios and companion for consistency reasons. My preference is 3a. I offer to implement 3. or 3a. |
Thank you very much for this comprehensive answer. I have a much better understanding of the problem now, |
"...or completely give up the possibility to choose the RSSI-Source and always use the default." This is what I had in mind with 3a. |
...In this case, Solution 3a is absolutely ok for me (and probably 99.9 % of the users) |
@pfeerick I'd appreciate your feedback. And for the records: there is a workaround for users caught in the rssiSource trap when migrating from OTX to ETX on color radios. Edit modelxx.yml, find the rssiSource: entry and change it to rssiSource: none. |
My take based on what I'm reading from that discussion from discord, and from the discussion here is that it is regretted that the option was ever added as it was done on the promise of features that never eventuated, and has simply led to confusion. So I would agree with 3a's take... reset it to the default (if I understand correctly, dump the setting entirely), and rip it out of the B&W and companion UIs. If there is an edge case that makes sense, well, someone can open an issue so we know it might actually be a desirable option to have in a future version. FYI, don't hurry on this one, don't really intend to include it in 2.9, so leave it a week or so before doing the PR if you still want to do it ;) |
Aye, Aye, Captain |
- remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion
- remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion
- remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion
* fixes #2552 - remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion * deleted rssiSource functional code Companion, simulator and radio * forgot one commented line
* fixes #2552 - remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion * deleted rssiSource functional code Companion, simulator and radio * forgot one commented line
commit 865d34a Author: 3djc <[email protected]> Date: Sat Jun 24 11:11:43 2023 +0200 'oh boy' fix commit c15cbf3 Author: 3djc <[email protected]> Date: Thu Jun 22 10:00:34 2023 +0200 yaml update commit a31e1e9 Author: 3djc <[email protected]> Date: Thu Jun 22 08:52:45 2023 +0200 reuse ! commit d28be2d Author: 3djc <[email protected]> Date: Thu Jun 22 08:39:32 2023 +0200 yaml update commit 7482fee Author: 3djc <[email protected]> Date: Wed Jun 21 14:30:14 2023 +0200 Fix possible muted on boot scenario commit e51f686 Author: 3djc <[email protected]> Date: Wed Jun 21 12:27:37 2023 +0200 compil fixes commit fbde97f Author: 3djc <[email protected]> Date: Wed Jun 21 12:18:11 2023 +0200 Some compile fixes commit fe08f71 Author: 3djc <[email protected]> Date: Wed Jun 21 11:12:19 2023 +0200 User configurable audio mute commit 5268ba0 Author: Peter Feerick <[email protected]> Date: Sun Jun 25 20:37:41 2023 +1000 chore: GH should be building 2.9 branch commit 0f0e71e Author: Peter Feerick <[email protected]> Date: Sun Jun 25 20:02:14 2023 +1000 chore: Add codename commit 5ebd090 Author: philmoz <[email protected]> Date: Sun Jun 25 15:09:46 2023 +1000 fix: Rounding errors in ADC value calculation for 6POS switch (#3713) commit 3ea236e Author: Peter Feerick <[email protected]> Date: Fri Jun 23 18:46:06 2023 +1000 fix(sim): do not allow launching if no radio profiles found (#3712) * Test for existence of radio profiles and advise if not found * Fix message box title and move simulator name to constants commit 41467e3 Author: Michael <[email protected]> Date: Fri Jun 23 05:52:54 2023 +0200 chore: Remove rssiSource and use default value (#3643) * fixes #2552 - remove edit option rssiSource - reset rssiOption to default (none) if carried over from OpenTX conversion * deleted rssiSource functional code Companion, simulator and radio * forgot one commented line commit 7b509a7 Author: philmoz <[email protected]> Date: Fri Jun 23 10:37:12 2023 +1000 fix(cpn): Wrong input offset value saved to YAML if GV (#3707) commit cee7393 Author: Neil Horne <[email protected]> Date: Fri Jun 23 09:07:55 2023 +1000 feat(cpn): Add radio sort order edit and sorting models list by any column (#3659) * feat(cpn): Add radio sort order edit (color) and sorting models list by any column (all) * Add column double click to enable/disable model sorting (disabled by default) commit 470db43 Author: Michael <[email protected]> Date: Thu Jun 22 12:34:09 2023 +0200 fix(color): Firmware update progress text stuck at device reset (#3704) * fixes #3695 flash progress * move init() to where it belongs (constructor) commit 302bf3e Author: richardclli <[email protected]> Date: Thu Jun 22 13:14:12 2023 +0800 fix: 3POS switches on certain radios not detecting all switch positions (#3568) * Quick and dirty fix for #3560. * Fixed build error. * Fixed problematic logic. * Code cleanup. commit a1528db Author: richardclli <[email protected]> Date: Wed Jun 21 18:56:55 2023 +0800 chore: Add EL18 specific build for v2.9 (#3539) * chore: Add EL18 specific firmware naming (#2609) * Added EL18 specific firmware naming. * fix: Update USB identifier * chore: Add to github builds * fix: Lowercase s in Flysky * chore: Reference EL18 specific firmware file Co-authored-by: Peter Feerick <[email protected]> * chore(el18): Change default switch cfg to match production (#2675) * chore: no nightly build for this branch commit 72c733c Author: 3djc <[email protected]> Date: Tue Jun 20 11:32:12 2023 +0200 fix: (re)enable time based mute for TX16S based on RM feedback (#3701)
Is there an existing issue for this feature request?
Is your feature request related to a problem?
Whenever something changed in my telemetry the RSSI telemetry source gets lost and on the field, I cannot select it - I can only do this in the Companion and go to tlemetry and select "RSSI()". Not being able to select this in the TX ends in repeating "RSSI critical" voice alarms, which leads to switching it off - bad idea.
Describe the solution you'd like
Being able to use the same dropdown menu in the TX that I can use to select RSSI telemetry source in the telemetry menu.
Describe alternatives you've considered
Set it in the Companion - but on the field useless.
Switch Voice alarms off
Additional context
No response
The text was updated successfully, but these errors were encountered: