search

pimatic / rfcontroljs

nodejs modul with protocol support for different 433mhz switches and weather stations for the RFControl Arduino library.
GNU General Public License v3.0
49 stars 54 forks source link

Changed pulseLengths #17

Closed incmve closed 9 years ago

incmve commented 9 years ago

pulseLengths seemed to be off, changed it and confirmed working received: [ 490, 2013, 4021, 9052 ]

Icesory commented 9 years ago

Nice. I failed this xD

incmve commented 9 years ago

@Icesory I think the battery indication was wrong. I'm always getting battery: 1 but according to the documentation 1 is almost empty battery. 0: Sensor's battery voltage is normal. 1: Battery voltage is below ~2.6 V.

I tried this commit localy and it seems to work but I'm not sure cause I have no drained battery's :( weather5: { id: 7, battery: 0, rain: 1.5 } Was there a particular reason for the 1-?

Icesory commented 9 years ago

Yes I have turned the battery indication. So a 1 is equal to true and true is good. This was it in my mind. Feel free to change this.

incmve commented 9 years ago

No that actualy makes more sense, didn't look at it that way lol :+1: