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

Feature Request: Don't update the thumbnail #39

Open
yphoenix opened this issue Dec 20, 2020 · 10 comments
Open

Feature Request: Don't update the thumbnail #39

yphoenix opened this issue Dec 20, 2020 · 10 comments
Labels
needs clarification It isn't clear if this is a bug, or human error. More details are needed in order to triage

Comments

@yphoenix
Copy link

I like my thumbnails, I set them up just how I want them.
I don't like them being randomly changed whenever motion is detected.
Please can we have an option to not update the thumbnail.
Config in the config file will work for me so you don't need to build more UI, or if you do on a system by system basis (I have two systems).
Thanks

@hunterpaul
Copy link

+1 for this feature. I’m really picky about my thumbnails and get annoyed when they change

@yphoenix
Copy link
Author

yphoenix commented Jan 2, 2021

Note: If you give me some pointers I'm happy to do the leg work and give you a pull request for the change.

@colinbendell
Copy link
Owner

it would be helpful for me to understand the flow you'd like to see. Can you clarify why you wouldn't want the thumbnail to update when motion is detected? If you don't want the thumbnails updated, do you really want the camera exposed? Would it be better to just have the motion sensor then since the thumbnails won't have meaningful new data? Or do you just want the live view?

I'm trying to extrapolate what the use case is here where you want a camera that doesn't act like a camera...

@colinbendell colinbendell added the needs clarification It isn't clear if this is a bug, or human error. More details are needed in order to triage label Feb 21, 2021
@yphoenix
Copy link
Author

Sure. The thumbnail is one that I have chosen - for whatever reason (eg: room is tidy, well lit, etc). Thus when using the Blink app to review video or alter the Blink settings for the camera I recognise which camera is which by the thumbnail. In one of the thumbnails (over the frontdoor) it was of myself and my wife a few days after we moved in. I liked seeing that thumbnail whenever I used the Blink app, now it is gone.

Really I used the Homebridge interface to enable / disable whole sets of cameras (I have two different setups at home) on an automatic basis - like when the last person leaves home etc. I don't need it to notify me that motion has been detected as the Blink app already does that, but I don't mind if it does. Viewing video through the HomeKit interface doesn't seem to work anyway and I always go to the Blink app when there is a notification anyway.

So really my functionality is System Alarm / Disarm plus some useful information on each camera like: temperature, motion detection on / off, battery level.

Obviously others have other requirements, extra stuff is a bonus.

Thanks for all your hard work.

@hunterpaul
Copy link

My use case is exactly the same. The plugin allows me to set up automations for arming/disarming individual cameras at specific times or arming the system when I leave home. I chose the thumbnails so that they are easily recognisable. When they are changed automatically to say night vision versions, some of the cameras aren’t as easily identifiable. For me, it’s the automation side of things that make this such a great plugin so being able to disable the bits I don’t need would make it even better and more “lightweight”

@jackflem1
Copy link

did this go anywhere, I would like the option to just arm / disarm the system. I don't need see the camera's from within the home app..

@JorgenVonStrangle82
Copy link

Another +1 for having the option to leave thumbnails as I set them.

@colinbendell
Copy link
Owner

Interim solution (next release) will be to set the poll interval to -1. A better solution is in the works.

@yphoenix
Copy link
Author

Thanks for working on this.

@JorgenVonStrangle82
Copy link

Sweet, thanks for working on this one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs clarification It isn't clear if this is a bug, or human error. More details are needed in order to triage
Projects
None yet
Development

No branches or pull requests

5 participants