Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Enhanced how Arlo queries API to reduce polls #75

Merged
merged 14 commits into from
Jun 5, 2018
Merged

Conversation

tchellomello
Copy link
Owner

@tchellomello tchellomello commented Jun 2, 2018

  • Implemented cached video list to reduce visits to the API
  • Implemented top level PyArlo.update() method to reduce loop in all cameras
  • Implemented method to return datetime object type for each ArloVideo object

Related issue: home-assistant/core#13176

WIP Home Assistant Future PR: tchellomello/home-assistant@dev...arlo_fix

With that Home Assistant future PR (depends on this PR to be approved) I got significaly improvements on the Arlo component. It would be great if you could test that too.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 79.743% when pulling 9f00765 on arlo_fixes into ff8aca2 on master.

@coveralls
Copy link

coveralls commented Jun 2, 2018

Coverage Status

Coverage decreased (-1.4%) to 79.577% when pulling 0134833 on arlo_fixes into ff8aca2 on master.

@tchellomello
Copy link
Owner Author

@jwillaz @viswa-swami @broox @ryanwinter can I have your guys opinion?

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

Successfully merging this pull request may close these issues.

2 participants