search

skrollme / homebridge-eveatmo

Homebridge plugin which adds a Netatmo weatherstation as HomeKit device and tries to act like Elgato Eve Room/Weather
Apache License 2.0
69 stars 7 forks source link

Illegal Characteristic Supplied on Homebridge 1.3.0 Beta #56

Closed Marfre888 closed 3 years ago

Marfre888 commented 3 years ago

Describe the bug As Homebridge log states, firmware characteristic is supplied as a number not string.

Version you use

To Reproduce Upon restarting Homebridge with 1.3.0Beta

error

battery

mrmaximas commented 3 years ago

same issue

Marfre888 commented 3 years ago

evatmo

ghost commented 3 years ago

Same here too.

DJay-X commented 3 years ago

Public of Homebridge v1.3.0

[20.2.2021, 11:45:42] [homebridge-eveatmo] This plugin generated a warning from the characteristic 'S2W2': SET handler returned write response value, though the characteristic doesn't support write response! See https://git.io/JtMGR for more info. [Zuhause (Wohnzimmer) Balkon@Zuhause (Wohnzimmer) Balkon History@S2W1] SET handler returned write response value, though the characteristic doesn't support write response! [20.2.2021, 11:45:42] [homebridge-eveatmo] This plugin generated a warning from the characteristic 'S2W1': SET handler returned write response value, though the characteristic doesn't support write response! See https://git.io/JtMGR for more info.

skrollme commented 3 years ago

Looks like I have to update to 1.3 to check and fix this... :D

skrollme commented 3 years ago

I fixed the firmware- and hopefully the battery-characteristics warning although the later does not showed up in my logs. When do S2W1 and S2W2-warnings show up in your logs @DJay-X ? On Startup, every now and then, while fetching the data (via EVE.app)? I haven't seen them so far either.

ghost commented 3 years ago

@skrollme when can we expect the update and test if there are any further issues? :)

skrollme commented 3 years ago

I want to wait for the reply of DJay concerning his problem. As far as I understand the plugin works as it did before, there are just some warning log entries now. So no need to hurry

mrmaximas commented 3 years ago

after update HB to 1.3.0 release no warnings appear

DJay-X commented 3 years ago

I want to wait for the reply of DJay concerning his problem. As far as I understand the plugin works as it did before, there are just some warning log entries now. So no need to hurry

Thanks @skrollme I waited yesterday and today but was not able to reproduce the log entry for whatever reason. So we are on the same page, nothing critical I guess. Thanks again.

skrollme commented 3 years ago

update 0.6.3 was released an hour ago and is already running on my setup. So far most of the warning are gone. But now the S2W1 and S2W2-warnings showed up in my logs too, @DJay-X :D It will take a look at it soon and publish another release when I found a fix for this.

ghost commented 3 years ago

update 0.6.3 was released an hour ago and is already running on my setup. So far most of the warning are gone. But now the S2W1 and S2W2-warnings showed up in my logs too, @DJay-X :D It will take a look at it soon and publish another release when I found a fix for this.

Same here, I got the warnings now too ^^

PS: I am using V1 from Netatmo Weatherstation, with main indoor module, secondary indoor module and one outdoor module, if that helps to evaluate where theese warnings are coming from.

skrollme commented 3 years ago

Looks like the S2W1 and S2W2-warnings come from the underlying fakegato-plugin, which is used to handle the history-functionality. I updated homebridge-eveatmo to the latest version if fakegato (0.6.1) and just published another release 0.6.4. Because the S2W*-warnings come not from the code of homebridge-eveatmo, it looks like there is nothing more I can do about it.