Enhancement: Add history to weather sensors about homebridge-weather

[werthdavid]

I can try to include it but won't be able to test it

[honkmaster]

Would be really appreciated. I will test it surely, additionally I'm looking into node.JS myself. Next time I will submit a PR :).... Greetings.

[werthdavid]

Can you try it with the latest version (1.6.0) and let me know if it works?

[honkmaster]

It does not seem to work. I used:

"pollingInterval": "1",
"enableHistory": true

Maybe there is an error in line 186. I don't know if temperature: temperature is valid, as the sample uses temp: , see:

this.loggingService.addEntry({time: moment().unix(), temp:this.temperature, pressure:this.airPressure, humidity:this.humidity});

Maybe you can compare with homebridge-dht: https://github.com/NorthernMan54/homebridge-dht/blob/master/index.js. This is working in Eve.

[werthdavid]

Good point... maybe now?

[honkmaster]

It is working. Thanks. One minor issue, the first value for temp is zero degrees.

[honkmaster]

Same is true after a restart of homebridge (for both tamp and humidity). Maybe only report valid data from cache.

[werthdavid]

Should be fixed with 1.6.2

Cool feature actually!

[werthdavid]

@honkmaster could you please test if it still works with 1.6.3? Don't know how homebridge behaves with potential undefined values...

[honkmaster]

There is no initial jump in the graph, seems to work. If you want, i can provide a screenshot after logging has taken place for a few days ...

[werthdavid]

Yes please, that would be cool! Thanks

[werthdavid]

Got one 😉 But thanks!
Really cool feature!