search

luc-ass / homebridge-evohome

Homebridge plugin for Honeywell Evohome
29 stars 16 forks source link

I miss the reporting of changes to current temperature #146

Open PuzzledUser opened 1 year ago

PuzzledUser commented 1 year ago

I know you decluttered the logging recently, but I miss seeing current temp changes. I used to consult the log now and again (e.g. to see why was it cold this morning when I woke up). I see you have various options in Settings. Would you consider adding one for log changes of current temperature?

It could enable code like this I've hacked in around line 460:

 this.log(
     device.name +
     " temp " +
     (oldCurrentTemp < newCurrentTemp ? "increased from: " : "decreased from: ") +
     oldCurrentTemp +
     " to: " +
     newCurrentTemp
 );

P.S. why does Evohome fake a change to current temperature when the target changes (past the current)? 🤔

PuzzledUser commented 1 year ago

By the way, did you accidentally overlook changing the logging to debug at line 916 (too wordy) and 969 (there is another log entry for that change a few seconds later)? The associated ones are now that.log.debug...