Claudio Ruggeri - CR-Tech
Claudio Ruggeri - CR-Tech
I didn't analyze the callgrind dump, but into the profile log I see a lot of warnings about `scan_interval is lower than 5 seconds, which may cause Home Assistant stability...
> Meanwhile I have changed it to 5 sec hoping it will help somehow but it didn't - out of memory crashes now happen less frequently... Why did you state...
Just another idea.. Can you post the modbus spec of the device you are integrating? I want to check some info to verify my suspect.
Unfortunately the yaml matches the spec and my suspect was wrong. The only further suggestion is to try break down the problem in smaller pieces, in example focusing on sensors...
> Still an issue The new HA release 2025.1 includes an updated version of the pymodbus library. As possible, can you check if the leaks are still present?
> You need to switch your sensor from Meteorological to Astronomical. Exactly. It is not an issue of the integration.
Yes, in this case you have to fork the dev branch, then apply the changes, submit a PR that it should be referenced here, under the "additional documentation". You miss...
> Can someone help me with the tests? You cannot request a review if all the changes have not been satisfied and if you cannot run test, it means that...
> I moved the initialization of on/off_values to the right place, but I want to make sure that the job that is launched will pass the tests or not. I...
> > > I moved the initialization of on/off_values to the right place, but I want to make sure that the job that is launched will pass the tests or...