search

ThomDietrich / miflora-mqtt-daemon

Linux service to collect and transfer Xiaomi Mi Flora plant sensor data via MQTT to your smart home system, with cluster support 🌱🌼🥀🏡🌳
MIT License
610 stars 140 forks source link

Requirements mixup #128

Closed martinlindjarv closed 3 years ago

martinlindjarv commented 4 years ago

With default installation (according to manual) requirements.txt might be wrong...

Just git pull and sudo pip3 install -r requirements.txt i gety an error: miflora 0.6 has requirement btlewrap==0.0.8, but you'll have btlewrap 0.0.9 which is incompatible.

Tried with pip3 install --force-reinstall btlewrap==0.0.8 and everything works.

Later tried to update miflora to 0.7 in requirements and then got an error: miflora 0.7.0 has requirement btlewrap==0.0.10, but you'll have btlewrap 0.0.9 which is incompatible.

So updated btlewrap to 0.0.10 and got it working ... (only one run for 7 sensors)

Should requirements.txt be updated ?

ThomDietrich commented 4 years ago

Hey Martin! Thanks for the report. Would you like to create a PR for btlewrap==0.0.10 and miflora==0.7.0?

martinlindjarv commented 4 years ago

I'll try... lets see what happens :)

martinlindjarv commented 4 years ago

Updated to latest miflora (according to https://pypi.org/project/miflora/)