bbqkees / ems-esp-domoticz-plugin

Domoticz plugin for the EMS Wi-Fi Gateway with Proddy's EMS-ESP firmware
MIT License
7 stars 8 forks source link

"onMessage" failed "Nameerror" every time domoticz starts and setpoint in domoticz not working. #10

Closed TaliesinDS closed 4 years ago

TaliesinDS commented 4 years ago

I am getting these errors every start.

` 2020-02-25 18:36:56.674 Error: (EMS bus Wi-Fi Gateway) 'onMessage' failed 'NameError'.

2020-02-25 18:36:56.674 Error: (EMS bus Wi-Fi Gateway) ----> Line 961 in '/home/pi/domoticz/plugins/ems-esp-domoticz-plugin/plugin.py', function onMessage

2020-02-25 18:36:56.674 Error: (EMS bus Wi-Fi Gateway) ----> Line 900 in '/home/pi/domoticz/plugins/ems-esp-domoticz-plugin/plugin.py', function onMessage

2020-02-25 18:36:56.674 Error: (EMS bus Wi-Fi Gateway) ----> Line 113 in '/home/pi/domoticz/plugins/ems-esp-domoticz-plugin/mqtt.py', function onMessage

2020-02-25 18:36:56.674 Error: (EMS bus Wi-Fi Gateway) ----> Line 932 in '/home/pi/domoticz/plugins/ems-esp-domoticz-plugin/plugin.py', function onMQTTPublish `

Also setting the temperature in domoticz does not seem to work. Thermostat temperature and setpoint are correctly displayed in domoticz, and they update when i manually change my thermostat (moduline 200) but changing the setpoint in domoticz does not change the setpoint of the thermostat. The plugin does send "thermostat_cmd_temp1 = 19.5" to the correct mqtt topic but nothing seems to happen on the ems gateway end.

im using domoticz v4.11724 and EMS-ESP version 1.9.5b43.

bbqkees commented 4 years ago

Aside from the NameError which is a plugin problem, it seems the Moduline 200 is not yet correctly implemented in EMS-ESP itself. Can you open an issue on the EMS-ESP repository? You need to log a bit of data with the Gateway when changing the temperature so we can look up the correct bytes in the telegrams.

TaliesinDS commented 4 years ago

I did that a couple days ago and it is supported by EMS-ESP now.

bbqkees commented 4 years ago

Ok I'll close this one for now. The Nameerror will be resolved in a next update.