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

Option to include/exclude specific detections #23

Open
kavejo opened this issue Jan 4, 2025 · 4 comments · May be fixed by #28
Open

Option to include/exclude specific detections #23

kavejo opened this issue Jan 4, 2025 · 4 comments · May be fixed by #28

Comments

@kavejo
Copy link

kavejo commented Jan 4, 2025

Similarly to #11 I would suggest implementing a mechanism that allows filtering for detection type (i.e. car, person, dog, etc.). There might be circumstances where Frigate might be able to detect multiple objects but there might be a desire to only receive messages for a subset.

@Sidonai-1
Copy link

I agree, this would be very useful.

My use case would be to set it only to alerts, and not detections, for example.
That way all the config is already done in frigate.

@OldTyT OldTyT linked a pull request Feb 27, 2025 that will close this issue
@OldTyT OldTyT removed a link to a pull request Feb 27, 2025
@OldTyT OldTyT linked a pull request Feb 27, 2025 that will close this issue
@OldTyT
Copy link
Owner

OldTyT commented Feb 27, 2025

Hello everyone, I've prepared the changes (very promptly xD).
Please, test it:

docker pull ghcr.io/oldtyt/frigate-telegram:dev-amd64
docker pull ghcr.io/oldtyt/frigate-telegram:dev-arm32v7
docker pull ghcr.io/oldtyt/frigate-telegram:dev-arm32v6
docker pull ghcr.io/oldtyt/frigate-telegram:dev-arm64v8

fyi @kavejo, @Sidonai-1

@OldTyT
Copy link
Owner

OldTyT commented Feb 27, 2025

Variable Default value Description
FRIGATE_EXCLUDE_LABEL None List exclude frigate event, separate ,
FRIGATE_INCLUDE_LABEL All List Include frigate event, separate ,

@Sidonai-1
Copy link

Sidonai-1 commented Feb 28, 2025

Hello everyone, I've prepared the changes (very promptly xD). Please, test it:

I had to play youtube videos of dogs barking to test it, but it seems to be working nicely!! (I excluded 'dog' and left the 'include' as it was: bark, for example)

Thanks a lot : )

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

Successfully merging a pull request may close this issue.

3 participants