-
Notifications
You must be signed in to change notification settings - Fork 6
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
Search for extended EMS+ entities (switchPrograms and holidayModes) are disabled for now #98
Comments
Seems you are polling very fast, maybe flooding the TCP stack. Can you add more delay.
this is just 35ms between the two polls. The other isue are: with 2 thermostats it does not send the correct deviceID (in this case 0x10 is master) and sends NaN as deviceID. I'll check what ems-esp do with this. |
I will check .... I do have 1000 msec delay between 2 read requests
Yes the function is not able to work with more then 1 thermostat. I am thinking about disabling the function until we have a working solution within EMS-ESP to read the data by API get. |
I remember Michael adding the send/response logic because of ioBroker. But happy to remove it and replace it. Do you have any suggestions on how this could be implemented? Perhaps have EMS-ESP Push changes directly via SSE/WebSocket? Or integrate this into a system command which can be called in the Scheduler to POST the data to another IP address? |
@proddy 4.8.0 should be the next stable version of the ioBroker adapter - but without the extended search functions. The switchPrograms are discussed with @MichaelDvP in emsesp/EMS-ESP32#1594 . |
emsesp/EMS-ESP32#1935
has indicated that the search for extended entities might bring the EMS-ESP gateway to crash because of memory problems.
There is no solution until now - I have disabled this function within ioBroker adapter until a better solution is available.
P.S.: It will continue to work for those users who are using it today.
The text was updated successfully, but these errors were encountered: