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

NEW: Add T8600 (E330) #408

Merged
merged 5 commits into from
Jan 2, 2024
Merged

Conversation

martijnpoppen
Copy link
Contributor

@martijnpoppen martijnpoppen commented Nov 18, 2023

Added the E330 professional camera
Couldnt test live streaming

Unfortunately i dont have this device. Got access via the Homey forum.

@bropat bropat self-assigned this Nov 19, 2023
@bropat bropat changed the base branch from master to develop November 19, 2023 17:15
@bropat
Copy link
Owner

bropat commented Nov 19, 2023

Todo:

  • Add entry for device in docs/supported_devices.md
  • Add images for device under docs/_media
  • Test streaming

@PhilippEngler
Copy link
Contributor

One user of my add-on has started to test this implementation with the T8600. But he stops after he could not switch on/off the "person detection" feature in the app anymore because it is disabled since the value has been changed by the add-on:

T8600_App

Do you have a solution how to get that back working?

@bropat bropat added work in progress Working on it and removed need hardware or access labels Dec 3, 2023
@bropat
Copy link
Owner

bropat commented Dec 3, 2023

@PhilippEngler

Face recognition and person recognition are connected. If you deactivate facial recognition, you can change the person recognition, otherwise not.

@PhilippEngler
Copy link
Contributor

Thank you for the answer, @bropat. I have shared that. This must be new, because for the cam 3 these values are not connected.

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

Successfully merging this pull request may close these issues.

3 participants