history argument animal_present, person_present, vehicle_present are not implemented

[ghost]

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]

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]

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

[tchellomello]

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. 👍

mmello

[ghost]

@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]

any updates on this?

[github-actions]

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.