Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Add a new ZbState called ZbLoaded to indicate that all ZbDevices and ZbLoads have been loaded successfully. #21305

Closed
tasict opened this issue Apr 30, 2024 · 5 comments · Fixed by #21320

Comments

@tasict
Copy link

tasict commented Apr 30, 2024

I hope to add a new state to indicate that all ZbDevices and ZbLoads have been loaded successfully.

@sfromis
Copy link
Contributor

sfromis commented Apr 30, 2024

I's say that it should be safe to use the existing {"ZbState":{"Status":0,"Message":"Started"}} with an extra one-second interval for the plugin data to be loaded from the local file system.

@tasict
Copy link
Author

tasict commented Apr 30, 2024

No, at this point, the ZB device and zb driver have not actually finished loading yet. If ZbState#Status=0 is added in the rule, calling Zbinfo at this time will fail to display information properly. It requires a delay of about 2 to 3 seconds before calling zbinfo. This is exactly why there is a need to add a new state in reality.

@s-hadinger
Copy link
Collaborator

Got it. I will add it

@s-hadinger
Copy link
Collaborator

Or move Started event after the plug-ins are loaded, which sounds better

@tasict
Copy link
Author

tasict commented Apr 30, 2024

Indeed, that would be perfect!

Repository owner locked and limited conversation to collaborators Apr 30, 2024
@Jason2866 Jason2866 converted this issue into discussion #21306 Apr 30, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants