Add sweep_frequency service to RemoteEntity #1076
Replies: 4 comments
-
I very much support this proposal. I had much difficulty in scanning my RF remote ceiling fan controllers though the HA UI remote learn service. Main problem was that my remotes are of the type where a long press (to keep transmitting for the frequency sweep) puts the remote into a different mode and the subsequent short press gets recorded as a different command to the one I was trying to record. I eventually gave up on using the HA UI learn approach and instead used the fixed frequency patch of the python-broadlink project to scan my remotes which worked much more reliably. One thing I'd like to add is that I would like to see HA UI have the option of being able to manually specify the frequency instead of calling the sweep service. I know all my remotes are 433.92MHz so I see no need to sweep unless that is just a once-off and the frequency can be saved for many subsequent learn service calls. |
Beta Was this translation helpful? Give feedback.
-
You are the man! Just got the new Broadlink and the HA integration is way slower than the original app. This would be the solution! Thank you! |
Beta Was this translation helpful? Give feedback.
-
This would be so incredibly useful. Please add it! |
Beta Was this translation helpful? Give feedback.
-
yes please! |
Beta Was this translation helpful? Give feedback.
-
Context
In the process of scanning commands from RF remotes, it's essential to first scan the frequency. Currently, both tasks are managed by the same service — learn_command — which presents usability challenges for learning RF commands due to the following reasons:
Proposal
A preferable approach involves separating these tasks into distinct services — one dedicated to identifying the frequency and another to learn the commands themselves. This separation of concerns offers several benefits, including:
Impact
Only two integrations have implemented the learn_command service so far:
The only one that supports learning RF commands is Broadlink, so it will be the only one impacted by this change.
Proof of Concept
home-assistant/core#115785
home-assistant/core#115791
Beta Was this translation helpful? Give feedback.
All reactions