Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
secabeen opened this issue Jul 23, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@secabeen
Copy link

secabeen commented Jul 23, 2023

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 secabeen added the bug Something isn't working label Jul 23, 2023
@GlennGoddard
Copy link
Contributor

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

@secabeen
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants