search

klejejs / python-thermia-online-api

A Python API for Thermia heat pumps using https://online.thermia.se
GNU General Public License v3.0
25 stars 10 forks source link

problem running version 3.4 #13

Closed wouterse closed 1 year ago

wouterse commented 1 year ago

After downloading, the latest version (3.4) the api stopped working for my code. Running the example gives the same error:

Traceback (most recent call last): File "example.py", line 2, in from ThermiaOnlineAPI import Thermia File "/home/pi/python-thermia-online-api-main-4/ThermiaOnlineAPI/init.py", line 3, in from ThermiaOnlineAPI.api.ThermiaAPI import ThermiaAPI File "/home/pi/python-thermia-online-api-main-4/ThermiaOnlineAPI/api/ThermiaAPI.py", line 28, in from ..model.HeatPump import ThermiaHeatPump File "/home/pi/python-thermia-online-api-main-4/ThermiaOnlineAPI/model/HeatPump.py", line 41, in "hot_water_boost_switch": None, TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

klejejs commented 1 year ago

Hmm, this is a weird error. Which Python version are you using?

wouterse commented 1 year ago

I'm running Python 3.7.3 (default, Jul 25 2020, 13:03:44), the api version that I downloaded 23/12 did not have this problem.

Op 29 dec. 2022, om 16:26 heeft Krisjanis Lejejs @.***> het volgende geschreven:

Hmm, this is a weird error. Which Python version are you using?

— Reply to this email directly, view it on GitHub https://github.com/klejejs/python-thermia-online-api/issues/13#issuecomment-1367409291, or unsubscribe https://github.com/notifications/unsubscribe-auth/AY4EXTJT67YES5UIS6FUXJDWPWUSBANCNFSM6AAAAAATMFPGPU. You are receiving this because you authored the thread.

klejejs commented 1 year ago

Sorry for the error, it was my mistake. In the latest release, I added typings for some variables and functions but did not think about how that syntax would affect older Python versions. It turns out the syntax was supported only by Python 3.10 and newer. The new release 3.5 has the issue fixed.

wouterse commented 1 year ago

Thanks for your quick fix!

Op 29 dec. 2022, om 18:07 heeft Krisjanis Lejejs @.***> het volgende geschreven:

Sorry for the error, it was my mistake. In the latest release, I added typings for some variables and functions but did not think about how that syntax would affect older Python versions. It turns out the syntax was supported only by Python 3.10 and newer. The new release 3.5 has the issue fixed.

— Reply to this email directly, view it on GitHub https://github.com/klejejs/python-thermia-online-api/issues/13#issuecomment-1367468865, or unsubscribe https://github.com/notifications/unsubscribe-auth/AY4EXTMMRYAWTIX5QT4XUILWPXAOBANCNFSM6AAAAAATMFPGPU. You are receiving this because you authored the thread.