hass-weatherflow2mqtt icon indicating copy to clipboard operation
hass-weatherflow2mqtt copied to clipboard

Won't start using docker-compose without defining ZAMBRETTI_MIN|MAX_PRESSURE

Open secabeen opened this issue 11 months ago • 2 comments

Home Assistant Version?

2023.6.2

Add-On or Self Hosted?

Self managed Container

Weatherflow2MQTT version?

3.1.6

What happened?

I just setup WeatherFlow2MQTT with a new docker compose up, and it was unable to start using the example docker-compose file. With ZAMBRETTI_MIN_PRESSURE and ZAMBRETTI_MAX_PRESSURE blank, it did not pull in the defaults, instead crashing with this error ValueError: could not convert string to float:

Defining the Zambretti pressures using the defaults from the main documentation allowed WeatherFlow2MQTT to start properly.

Relevant log output

ted-weatherflow2mqtt-1  |     sys.exit(load_entry_point('weatherflow2mqtt==3.1.6', 'console_scripts', 'weatherflow2mqtt')())
ted-weatherflow2mqtt-1  |   File "/usr/local/lib/python3.10/site-packages/weatherflow2mqtt-3.1.6-py3.10.egg/weatherflow2mqtt/__main__.py", line 10, in main
ted-weatherflow2mqtt-1  |   File "/usr/local/lib/python3.10/asyncio/runners.py", line 44, in run
ted-weatherflow2mqtt-1  |     return loop.run_until_complete(main)
ted-weatherflow2mqtt-1  |   File "/usr/local/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
ted-weatherflow2mqtt-1  |     return future.result()
ted-weatherflow2mqtt-1  |   File "/usr/local/lib/python3.10/site-packages/weatherflow2mqtt-3.1.6-py3.10.egg/weatherflow2mqtt/weatherflow_mqtt.py", line 798, in main
ted-weatherflow2mqtt-1  | ValueError: could not convert string to float: ''
ted-weatherflow2mqtt-1 exited with code 0

secabeen avatar Jul 23 '23 00:07 secabeen

Did you leave them blank or not include them. If you don't include them then there is a default value set.

GlennGoddard avatar Aug 30 '23 20:08 GlennGoddard

I did not include them at all; I was expecting the defaults to be used, but they were not.

secabeen avatar Aug 30 '23 20:08 secabeen