search

mark-wade / homebridge-hive

Hive Thermostat plugin for Homebridge
MIT License
12 stars 5 forks source link

Battery level #8

Closed PianSom closed 7 years ago

PianSom commented 7 years ago

Hi - I'm not sure your battery code is working properly just yet. I updated to 1.4.0 and restarted homebridge. I then went to my ios home app, and looked at the details for my two thermostats. In both cases I got a "Fetching data from Hive API" on the homebridge log.

But in both cases the ios app reports 100% battery (under Details for the thermostat). This seems unlikely, since my batteries are some 4 months old, and both the Hive web page and Hive ios app show graphics of both sets of batteries being less than full.

mark-wade commented 7 years ago

Hm... it seems to be working for me, I wonder if there is something wrong with multiple thermostats.

Could you get the output of /nodes and email it to me (m.wade@me.com)?

To do that, find this line:

var successHandler = function(body){

And add below:

console.log(body);
PianSom commented 7 years ago

sent

mark-wade commented 7 years ago

Try with 1.4.1 just published and let me know if the issue is resolved

PianSom commented 7 years ago

Both thermostats showing 80% now. Which is credible.

Though perhaps slightly out of kilter with the graphic on the ios app ...

img_2780

mark-wade commented 7 years ago

I think Hive's app uses that icon for 80 or below - mine is on 75% and looks like it's using the same icon.

PianSom commented 7 years ago

Hmm - odd.

I stopped homebridge and ran a sudo npm update -g homebridge-hive and then a sudo update-g homebridge (as I was out of date on 0.4.17).

I then tested and reported as above.

But I have just done a npm -g outdated and am seeing that I am still running homebridge-hive 1.4.0 and that 1.4.1 is available. I tried another update of homebridge-hive but it seems to have no effect. I'm not sure what is going on or what version I am actually running.

It may be worth logging the homebridge-hive version being run at startup