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

the code should use get_light() + get_sensor() even with OO access #109

Open
fingon opened this issue May 4, 2017 · 1 comment
Open

Comments

@fingon
Copy link

fingon commented May 4, 2017

Currently, if you use e.g.

get_{sensor,light}_objects

and then iterate through them, and do stuff, it is orders of magnitude slower due to built-in rate limiting (and number of HTTP requests) than if you just do once

get_{light,sensor}

and look at the full JSON dump (that has been around years, since bridge sw ver 1.3).

For example, in my case, I had to do following changes to get reasonable performance with 'few' lights/motion sensors/tap (and more coming up):

fingon/kodin-henki@d30a518

@robmarkcole
Copy link

+1 this should be reviewed

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

2 participants