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

Homematic only shows one rssi value #20900

Closed
dagobert opened this Issue Feb 9, 2019 · 0 comments

Comments

Projects
None yet
3 participants
@dagobert
Copy link
Contributor

dagobert commented Feb 9, 2019

I am using Homematic devices with HASS.
Sensors have only one RSSI attribute (RSSI_DEVICE) which is reflecting the (incoming) signal strength measured by the CCU of the sensor. A sensor only sends data.

Actors for example have two RSSI attributes. One is the same as with the sensors and the other one is indicating the signal strength measured by device (receiving data from the CCU).

The homematic component is collecting both RSSI Attributes but in the UI of Home Assistant only one attribute is visible and it is called rssi. This makes me believe that it is not correct of fully implemented. The rssi attribute in HASS should be called rssi_device and rssi_peer.

dagobert added a commit to dagobert/home-assistant that referenced this issue Feb 9, 2019

Fix home-assistant#20900: RSSI_PEER and RSSI_DEVICE are different things
This change is fixing issue home-assistant#20900.

Wireless actors are having two RSSI values. The way the component was programmed one of them was overwritten.

@wafflebot wafflebot bot added in progress labels Feb 9, 2019

@wafflebot wafflebot bot removed the in progress label Feb 18, 2019

andrewsayre added a commit that referenced this issue Feb 18, 2019

RSSI_PEER and RSSI_DEVICE are different things (fixes #20900) (#20902)
* Fix #20900: RSSI_PEER and RSSI_DEVICE are different things

This change is fixing issue #20900.

Wireless actors are having two RSSI values. The way the component was programmed one of them was overwritten.

* Added deprecation comment

* Fixed long line

* Fix: pylint comment

* Lint

* flake8

* flake8 again

* Update __init__.py

@balloob balloob referenced this issue Mar 6, 2019

Merged

0.89.0 #21712

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.