python-ring-doorbell icon indicating copy to clipboard operation
python-ring-doorbell copied to clipboard

history argument animal_present, person_present, vehicle_present are not implemented

Open ghost opened this issue 7 years ago • 6 comments

Here is output where kind=none

myring.doorbells[1].history(limit=1, timezone=None, kind=None, enforce_limit=False, older_than=None, retry=8) [{u'kind': u'ding', u'vehicle_present': None, u'person_present': None, u'created_at': datetime.datetime(2018, 5, 24, 22, 39, 31, tzinfo=<UTC>), u'favorite': False, u'animal_present': None, u'events': [], u'recording': {u'status': u'ready'}, u'answered': False, u'id': 655924564564574686, u'snapshot_url': u''}]

Here is output where kind is other than none:

myring.doorbells[1].history(limit=1, timezone=None, kind=ding, enforce_limit=False, older_than=None, retry=8) Traceback (most recent call last): File "<stdin>", line 1, in <module> NameError: name 'ding' is not defined

Am I doing something wrong here.

What I would like to do is write a script to test and see if the person present, and animal present features work.

So what I was trying to do was filter by dings so that I know a person is present and see if it returns that as true. If so I want to write a script that checks if those values are true and then save those videos and run machinebox facebox on those videos.

I am able to save videos and manual running those videos through the machine learning and it recognizes the faces I have trained, now I want to automate that.

Any help would be appreciated.

Thanks

Jon

ghost avatar May 24 '18 23:05 ghost

I figure it out, it requires the kind=u'ding'

however it appears that ring is not running the machine learning on there video yet because people and vehicles are not returning values other than None.

ghost avatar May 25 '18 00:05 ghost

in case anyone else was wondering:

for event in frontdoor.history(limit=30): ... print('person_present %s' % event['person_present']) ... person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None person_present None

ghost avatar May 25 '18 00:05 ghost

Hello @darkstar939, I'm happy that you figured it out. Yes, the person_present and animal_present and vehicle_present are new attributes and are not yet incorporated in the history() method.

That is a nice feature to have and let's see if we can implement a future version. :+1:

mmello

tchellomello avatar May 26 '18 07:05 tchellomello

@tchellomello Do you know if its possible to enable advanced features and what they do?

u'features': {u'advanced_motion_enabled': False, u'motion_message_enabled': False, u'motions_enabled': True, u'night_vision_enabled': False, u'people_only_enabled': False, u'shadow_correction_enabled': False, u'show_recordings': True},

ghost avatar May 26 '18 07:05 ghost

any updates on this?

ghost avatar Feb 03 '19 04:02 ghost

There hasn't been any activity on this issue recently. This issue has been automatically marked as stale because of that. It will be closed if no further activity occurs. Please make sure to update to the latest ring_doorbell version and check if that solves the issue. Thank you for your contributions.

github-actions[bot] avatar Apr 18 '24 18:04 github-actions[bot]