-
Notifications
You must be signed in to change notification settings - Fork 17
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
Detected blocking call to import_module inside the event loop by custom integration 'zha_toolkit' #224
Comments
Thank you for reporting this. It seems that HA now added a requirement to do imports inside a task. home-assistant/core#116425 (comment) I have other priorities to handle. If somebody wants to fix and and propose a PR, that'ld be welcome. |
I don't have the skills to do so, sorry... |
Yes same warning all day |
In the mean time, I think that the log level for homeassistant.util.loop] could be set to "error" to stop filling the logs. |
Same issue keep on Core v2024.5.1 |
I would love to help but haven't the required knowledge. |
Can we still hope for a new version in the coming weeks? If I could have helped you, I would have :) |
What I don't get considering how important this tool is for some ZHA integrations. Why is ZHA not taking this tool and integrating it into the ZHA main repo. Any reasons they don't? This tool is a great addition to make ZHA even better. So the ZHA owners could maintain it? But yeah maybe there is a reason for that and I don't understand that. |
It's a custom integration. HA core Devs generally don't adopt custom integrations like that. Instead the author elects it for inclusion if they wish to. But realize that adds extra requirements to the Devs time and many custom integration authors generally aren't interested. If the author has no intention to do so - HA won't add extra work for themselves... They're already manpower constrained. |
The big problem is that the ZHA integration is missing functionality like reading in entity "not exposed" Cluster / Attribute. |
Then open a bug against zha to get the. To add the functionality. Slurping up a custom integration isn't how functions get tagged to HA. |
No for additions it's a feature requests, not a bug. |
The subject of the topic "Detected blocking call to import_module inside the event loop by custom integration 'zha_toolkit'" is a bug. |
@CordoWEB my reply was on having ZHA toolkit functions added to ZHA, that is a feature request. This issue is deprecation. Little harsh to call it a bug, but you are right (it's more maintenance). |
Thanks a lot to @benschumacher for his work. |
@mdeweerd can you push this update so it shows up via HACS? |
Note. Next on need to be fixed too. Open calls of files.
|
You have this error when you call what service ? |
Just startup. (New warning in beta of home Assistant) |
I do not have this error with HAS 2024.5.x. |
Its in 2024.6.0 beta |
@elupus I guess this issue won't be fixed in core, it requires a new toolkit version with the patch as provided by |
@elupus : When you'll add your new issue, could you make the same test when HA 2024.6.0 final version will be available ? |
Probably not. I dont use the integration anymore. So ive disabled it. Just wanted to give a heads up . The warning will guaranteed remain in final version (im part of ha member team) |
Core v2024.5.0
Every call to zha_toolkit generates this error:
The text was updated successfully, but these errors were encountered: