bombadiltom / homebridge-rademacher-homepilot

MIT License
26 stars 21 forks source link

Request: reduce number of log entries #54

Closed ebreg closed 4 years ago

ebreg commented 4 years ago

Hi Tom,

first of all thank you very much for the great plugin - it works like charm. I do have a suggestion for improvement, however:

I am using the contact sensor and smoke alarm. If the status of these is updated every few (10) seconds, the log is flooded with many entries (see below). Is it possible to disable those log entries? Or to write only a single-line entry in the log when a value changes?

Contact sensor:

[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür - [1010012] updating
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - getting current door state
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - getting current battery level
[6/22/2020, 21:26:57] [RademacherHomePilot] { contact_state: 'open' }
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - door is open = true
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - updating to is open = true
[6/22/2020, 21:26:57] [RademacherHomePilot] { contact_state: 'open' }
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - battery status = 72
[6/22/2020, 21:26:57] [RademacherHomePilot] Terassentür [1010012] - updating battery level to 72

Smoke Alarm:

[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer - [1010006] updating
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - getting smoke detected
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - getting current battery level
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - smoke detected=false
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - smoke detected = 0
[6/22/2020, 21:26:58] [RademacherHomePilot] { smoke_detected: false }
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - battery status = 100
[6/22/2020, 21:26:58] [RademacherHomePilot] Wohnzimmer [1010006] - updating battery level to 100
bombadiltom commented 4 years ago

Yeah, that is annoying... think I will add a debug switch soon...

ebreg commented 4 years ago

Thanks for the update, highly appreciated.