valentinfrlch / ha-llmvision

Let Home Assistant see!
Apache License 2.0
285 stars 12 forks source link

Support for Frigate Reviews topic and Customizable importance analysis prompt, fix 404 #94

Open jukkapirinen opened 1 week ago

jukkapirinen commented 1 week ago

Changes:

  1. Fix for 404 errors by ensuring clip exists before processing
    • Fixed in frigate/eventshandling
  2. Customizable importance analysis prompt.
    • Possible to edit the prompt what is sent if importance analysis enabled
  3. Support for Frigate Reviews MQTT topic (/frigate/reviews)
    • Handle different JSON structure for reviews topic
    • Filter events by severity (detection/alert)

Note: Not yet much tested so created Draft PR. Feedback welcome on any improvements or issues found.

valentinfrlch commented 1 week ago

Thank you for your PR, much appreciated! I didn't even know about frigate/review. Will take a look at this in the coming days. Do I need to enable anything in Frigate for this to work?

jukkapirinen commented 1 week ago

Works with frigate 0.14. It is bit confusing between all the settings now. The input selectors descriptions should be still be worked on and how to arrange them.