search

bramstroker / homeassistant-powercalc

Custom component to calculate estimated power consumption of lights and other appliances
MIT License
937 stars 250 forks source link

Support Python 3.10 for the measure tool #747

Closed bramstroker closed 2 years ago

bramstroker commented 2 years ago

The following library causes issues: https://github.com/CITGuru/PyInquirer. See if we can use https://github.com/magmax/python-inquirer.

bramstroker commented 2 years ago

Work in progress #780 This is working now in both Python 3.9 and 3.10. Not tested in 3.8 yet, but I think this will also work fine.

Needs a littlebit more monkey testing before I will merge.

bramstroker commented 2 years ago

Also resolved the resume logic error ValueError: invalid literal for int() with base 10: 'bri' @RubenKelevra. I also got that one while testing. The reason was only the header row was written to the CSV, and I did not check for that case yet (Only zero sized CSV)

bramstroker commented 2 years ago

Did quite a lot of testing and also made some other tweaks and improvements.

Has been merged into master now. Would be nice if you can verify everything is working correctly for you as well @RubenKelevra . The docker image also uses Python 3.10 now, previously the docker image was based on 3.8

RubenKelevra commented 2 years ago

Would be nice if you can verify everything is working correctly for you as well @RubenKelevra .

Works fine for me (outside of docker). The docker version was outdated at the time I checked – probably just a delay on the hub, I guess.