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

Frigate Blueprint Fix and Update #232

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lich2000117
Copy link

Tested, stable Frigate blueprint template update.

Fixed some frigate stability issues (Frigate video not ready etc)
0. added checks to ensure snapshot and clips are ready.

  1. update "tap_navigation", to default to video links, when user clicks on notification, it should now navigate user to the video captured.
  2. use "thumbnail" instead of snapshot in notifications to reduce the chances of missing snapshot (403 or 404 error)
  3. long press the notification will show the high-res snapshot instead (again, minimise instability when sending out a notification, but image is broken)
  4. increase retry delay and times since Frigate may be struggling finalising videos

LLM compatibility:
5. remove "default" llm model so it would work better with Gemini

@lich2000117
Copy link
Author

Happy to reformat it a bit as clearly it has lots of syntax changes (single double quotation marks etc)

@valentinfrlch
Copy link
Owner

Thanks for your work!
However, the blueprint has been restructured a lot for v1.4.0; It doesn't support input_mode anymore. It is still compatible with frigate, but it doesn't fetch clips from frigate anymore and is no longer triggered by mqtt. Instead users should input the binary_sensor entities exposed by the Frigate integration as triggers.

The reason for this change is that there were a lot of issues especially with fetching the clips because the clip wasn't ready on Frigate's end by the time the mqtt event was sent.

You can check out the latest version of the blueprint (currently in beta) here: https://github.com/valentinfrlch/ha-llmvision/blob/main/blueprints/event_summary_beta.yaml

Some of the other changes still sound great and still apply. If you wan't to add some of these changes to the new blueprint, please commit to event_summary_beta.yaml.

Thanks again for all your work!

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 this pull request may close these issues.

2 participants