Skip to content
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

Open
tp1de opened this issue Aug 14, 2024 · 4 comments

Comments

@tp1de
Copy link
Owner

tp1de commented Aug 14, 2024

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.

@MichaelDvP
Copy link
Contributor

Seems you are polling very fast, maybe flooding the TCP stack. Can you add more delay.

2024-08-14 08:21:51.109 DEBUG 345: [command] Called command system/response (Hole Antwort)
2024-08-14 08:21:51.144 DEBUG 346: [command] Called command system/response (Hole Antwort)

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.

@tp1de
Copy link
Owner Author

tp1de commented Aug 14, 2024

Seems you are polling very fast, maybe flooding the TCP stack. Can you add more delay.

I will check .... I do have 1000 msec delay between 2 read requests

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.

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.
(no send / response logic anymore)

@tp1de tp1de changed the title Search for extended EMS+ entities (switchPrograms and holidayModes) might bring EMS-ESP gateway to crash Search for extended EMS+ entities (switchPrograms and holidayModes) are disabled for now because it might bring EMS-ESP gateway to crash Aug 27, 2024
@tp1de tp1de changed the title Search for extended EMS+ entities (switchPrograms and holidayModes) are disabled for now because it might bring EMS-ESP gateway to crash Search for extended EMS+ entities (switchPrograms and holidayModes) are disabled for now Aug 27, 2024
@proddy
Copy link

proddy commented Sep 1, 2024

I am thinking about disabling the function until we have a working solution within EMS-ESP to read the data by API get.
(no send / response logic anymore)

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?

@tp1de
Copy link
Owner Author

tp1de commented Sep 1, 2024

@proddy
I have disabled the search for switchPrograms and holidayModes within my last ioBroker adapter version, because I want to upgrade the stable version soon. Most ioBroker users are still using the stable adapter version 3.5.0 but have installed dev versions of EMS-ESP.
This combination braught quite some difficulties and instabilities. I need to think how to minimize my efforts to keep the ioBroker adapter alive and working with different firmware versions.
Maybe stable ioBroker versions should only work with stable EMS-ESP versions? (dev versions to be rejected??)

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 .
Michael is making good progress to implement the function within EMS-ESP.
When this ready and tested I will implement within ioBroker adapter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants