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

Not all open windows are recognized #209

Closed
CKMartens opened this issue Jan 21, 2021 · 7 comments
Closed

Not all open windows are recognized #209

CKMartens opened this issue Jan 21, 2021 · 7 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@CKMartens
Copy link

Since version 2.x all open windows are no longer recognized. I use HM IP twist grip sensors (HmIP-SRH) and HM IP window sensors (HmIP-SWDO) on the windows.

The opening of a window is recognized in the "Kitchen" room, but the second window is not. Rotary handle sensors are built into both.

Log:

heatingcontrol.0	2021-01-21 09:38:41.852	debug	(7491) Change Status WindowOpen in Küche to true
heatingcontrol.0	2021-01-21 09:38:41.852	debug	(7491) result for AddSensorState in Küche is true
heatingcontrol.0	2021-01-21 09:38:41.852	debug	(7491) one window open in Küche alias.0.detektoren.kueche.drehgriff_fenster.WINDOWS_STATE
heatingcontrol.0	2021-01-21 09:38:41.852	debug	(7491) add new sensor state in Küche alias.0.detektoren.kueche.drehgriff_fenster.WINDOWS_STATE 1
heatingcontrol.0	2021-01-21 09:38:41.852	debug	(7491) Küche ### ChangeStatus Sensor to {"OID":"alias.0.detektoren.kueche.drehgriff_fenster.WINDOWS_STATE","state":1} in auto

heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) ChangeStatus called, while already changing, push to list Küche Sensor {"OID":"alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE","state":2} (2)
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) handle sensor auto false 2 0 0
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) ChangeStatus called, while already changing, push to list Wohnzimmer Sensor {"OID":"alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE","state":2} (1)
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) handle sensor auto false 2 0 0
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) UpdateDPs for Wohnzimmer auto
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) result for AddSensorState in Wohnzimmer is false
heatingcontrol.0	2021-01-21 09:40:32.164	debug	(7491) add new sensor state in Wohnzimmer alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE 2
heatingcontrol.0	2021-01-21 09:40:32.163	debug	(7491) Wohnzimmer ### ChangeStatus Sensor to {"OID":"alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE","state":2} in auto

grafik
grafik
grafik
grafik

The same problem occurs in the "living room" room. No open window is recognized in the "bedroom" room.

In the "office" room, however, everything is recognized correctly.

My first guess was that the senor designation couldn't cope with spaces, but in the "office" space there are spaces in the designations as well.

@CKMartens CKMartens added the bug Something isn't working label Jan 21, 2021
@rg-engineering
Copy link
Owner

wurde doch schon hier #191 besprochen und sollte mit 2.0.1 erledigt werden.
Für 2.0.1 bitte direkt hier von github installieren wegen #208

@rg-engineering rg-engineering added this to the V2.0 Release milestone Jan 21, 2021
@rg-engineering rg-engineering self-assigned this Jan 21, 2021
@rg-engineering
Copy link
Owner

rg-engineering commented Jan 21, 2021

scheint doch ein anderes Problem zu sein.
meldet
alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE
true/false oder 2?

alias.0.detektoren.wohnzimmer.drehgriff_terrassentuer.WINDOWS_STATE 2
und
alias.0.detektoren.kueche.drehgriff_fenster.WINDOWS_STATE 1

dann würde das zu #146 passen...

@CKMartens
Copy link
Author

Die melden wie alle HM Fenstersensoren 0 für closed und 1 bzw 2 für open bzw tilted

@rg-engineering
Copy link
Owner

deshalb erkennt der Adapter Fenster offen in der Küche (1) aber nicht im Wohnzimmer (2)

bedeutet, geöffnete Fenster werden erkannt, gekippte also nicht...

muss ich umbauen...

@CKMartens
Copy link
Author

CKMartens commented Jan 21, 2021

Für die HmIP Drehgriffsensoren gelten 0=closed, 1=tilted und 2=open, für die HmIP Tür-/Fenstersensoiren 0=closed und 1=open. Ich habe es gerade geprüft, kippe ich die Fenster nur werden tatsächlich alle erkannt.

@rg-engineering
Copy link
Owner

ich habe die Logik umgedreht, anstatt auf value==true frage ich auf value !=false.
sollte also mit 2.0.2 dann passen...

@CKMartens
Copy link
Author

In 2.0.2 behoben. Es funktioniert wie es soll. Ich schließe das Issue.

Danke für Deine Arbeit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants