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

HMIP-eTRV battery/valve status not displayed #457

Closed
ThomasLoeffel opened this Issue Oct 29, 2018 · 4 comments

Comments

2 participants
@ThomasLoeffel

ThomasLoeffel commented Oct 29, 2018

Describe the bug

The new battery/valve display (new WebUI patch) in heating control devices is not visible for HmIP-eTRV.

To Reproduce

Steps to reproduce the behavior:
Go to 'Status und Bedienung'
Open a room with a HmIP-eTRV assigned to.
you will see no "Ventilstatus or Battery".

Expected behavior

Display valve state and battery status for HmIP-eTRV in 'Status und Bedienung'.

Screenshots

20181030 hmip-etrv

System information (please complete the following information):

RaspberryMatic 3.37.8.20181026
Hardware Asus-Tinker+
@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 30, 2018

After some investigation I found that at least the HmIP-eTRV does not allow to query the battery status and return it in volts. It only has a LOWBAT warning as it seems. However, I don't know for the eTRV2. So can you please use devconfig and try to display the Channel values for Channel :1 for the eTRV2 and list them here. Thanks.

jens-maus added a commit that referenced this issue Oct 30, 2018

reworked 0042-WebUI-HeatingControl-BatteryValveState to also add
Battery+Valve status for HmIP heating control devices. Note, however,
that due to missing BATTERY_STATE parameter in e.g. HmIP-eTRV this
information cannot be displayed compared to HM-CC-RT-DN. This refs #457.

@jens-maus jens-maus added this to the next release milestone Oct 30, 2018

@jens-maus jens-maus added this to In progress in WebUI improvements/fixes Oct 30, 2018

@ThomasLoeffel

This comment has been minimized.

ThomasLoeffel commented Oct 30, 2018

Hallo Jens,

ich habe die gewünschten Daten. Ich habe die Kanalliste mit einem Skript ausgelesen (ohne Werte) und dann mit ioBroker (mit Werten). Meiner bescheidenen Meinung nach kann man die Arbeitsspannung auslesen. Ich habe die beiden Positionen "gelb" markiert. Nach Batterie Wechsel ging die ausgelesene Spannung auf 3.0 (vorher 2,6V siehe Bild ioBroker) also 2*1,5V Mignon.

Skript: Wert eines HMIP-eTRV2 auslesen, hier Arbeitsspannung:
..............................................................
var ipserial = (datapoints.Get("HmIP-RF.Sernummer:0.OPERATING_VOLTAGE")).Value();
..............................................................

Danke Gruß Thomas

Anhang:

20181030 hmip-etrv2 skriptauswertung

Hier ein Auszug aus ioBroker mit Werten:

20181030 hmip-etrv2 iobroker

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Nov 1, 2018

@ThomasLoeffel Danke für die Auflistung. Allerdings liegt der OPERATING_VOLTAGE Datenpunkt in einem anderen Kanal :0 als bei den BidCos-RF Geräten und daher muss ich schauen inwieweit ich diesen Abgreifen kann da die Darstellung in der Geräteliste ja Kanalbasiert ist und der :1 Kanal der ja dargestellt wird diesen Datenpunkt faktisch nicht beinhaltet.

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Nov 5, 2018

Also den Status des Ventils konnte ich nun für die nächste version implementieren. Da der Wert der battery allerdings in Kanal :0 steckt habe ich mich dagegen entschieden nun etwas zu implementieren das in der Statusanzeige des Kanal :1 trotzdem die Batterieanzeige umsetzt. Ggf. sollte man solche Dinge besser in der Statusspalte unter Einstellunge->Geräte verankern oder in der Liste der Geräte direkt anzeigen. Definitiv aber nicht in Kanal :1

@jens-maus jens-maus closed this Nov 5, 2018

@jens-maus jens-maus moved this from In progress to Done in WebUI improvements/fixes Nov 5, 2018

jens-maus added a commit that referenced this issue Nov 5, 2018

changed StatusColumn WebUI patch to also display the OPERATING_VOLTAGE
status if present for a certain device (mostly HmIP). This refs #457.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment