gvigroux / hon

Support of all Haier, Candy, Hoover appliances integrated in the official hOn mobile app. Direct access to all possible services and parameters
MIT License
118 stars 45 forks source link

Error when logging in #76

Closed finalbillybong closed 1 year ago

finalbillybong commented 1 year ago

Hi when I try to add the intergration I get the following error.

Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 383, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/hon/__init__.py", line 74, in async_setup_entry await coordinator.device.load_commands() File "/config/custom_components/hon/device.py", line 182, in load_commands cmd = HonCommand(command, attr2, self._hon, self, multi=multi, program=program) File "/config/custom_components/hon/command.py", line 13, in __init__ self._ancillary_parameters = self._create_parameters(attributes.get("ancillaryParameters", {})) File "/config/custom_components/hon/command.py", line 23, in _create_parameters result[parameter] = HonParameterRange(parameter, attributes) File "/config/custom_components/hon/parameter.py", line 53, in __init__ self._min = int(attributes["minimumValue"]) ValueError: invalid literal for int() with base 10: '0,5'

gvigroux commented 1 year ago

Thanks, i will fix it soon!

gvigroux commented 1 year ago

Can you please activate the debug (left click on button [...] in the integration) and reboot HA ? One big DEBUG log starting by "command" will be useful for me

gvigroux commented 1 year ago

Should work now (I hope) :) I'm closing the ticket. please reopen if it's not working

nathan-gs commented 1 year ago

I have exactly the same error, with the latest commit (and 0.3.0). Any idea what's wrong @gvigroux

gvigroux commented 1 year ago

Are you sure you have the latest code ? I've made an update at this location. Can you please dump the error one more time?

gvigroux commented 1 year ago

Should not happens anymore

nathan-gs commented 12 months ago

Verified it works now with latest 0.5.2-Beta2

gvigroux commented 11 months ago

thanks