show negotiated mse codecs in console on error

.cspell/frigate-dictionary.txt

@@ -22,7 +22,6 @@ autotrack
 autotracked
 autotracker
 autotracking
-backchannel
 balena
 Beelink
 BGRA
@@ -192,7 +191,6 @@ ONVIF
 openai
 opencv
 openvino
-overfitting
 OWASP
 paddleocr
 paho
@@ -317,4 +315,4 @@ yolo
 yolonas
 yolox
 zeep
-zerolatency
+zerolatency

.github/DISCUSSION_TEMPLATE/beta-support.yml

@@ -1,129 +0,0 @@
-title: "[Beta Support]: "
-labels: ["support", "triage", "beta"]
-body:
-  - type: markdown
-    attributes:
-      value: |
-        Thank you for testing Frigate beta versions! Use this form for support with beta releases.
-
-        **Note:** Beta versions may have incomplete features, known issues, or unexpected behavior. Please check the [release notes](https://github.com/blakeblackshear/frigate/releases) and [recent discussions][discussions] for known beta issues before submitting.
-
-        Before submitting, read the [beta documentation][docs].
-
-        [docs]: https://deploy-preview-19787--frigate-docs.netlify.app/
-  - type: textarea
-    id: description
-    attributes:
-      label: Describe the problem you are having
-      description: Please be as detailed as possible. Include what you expected to happen vs what actually happened.
-    validations:
-      required: true
-  - type: input
-    id: version
-    attributes:
-      label: Beta Version
-      description: Visible on the System page in the Web UI. Please include the full version including the build identifier (eg. 0.17.0-beta1)
-      placeholder: "0.17.0-beta1"
-    validations:
-      required: true
-  - type: dropdown
-    id: issue-category
-    attributes:
-      label: Issue Category
-      description: What area is your issue related to? This helps us understand the context.
-      options:
-        - Object Detection / Detectors
-        - Hardware Acceleration
-        - Configuration / Setup
-        - WebUI / Frontend
-        - Recordings / Storage
-        - Notifications / Events
-        - Integration (Home Assistant, etc)
-        - Performance / Stability
-        - Installation / Updates
-        - Other
-    validations:
-      required: true
-  - type: textarea
-    id: config
-    attributes:
-      label: Frigate config file
-      description: This will be automatically formatted into code, so no need for backticks. Remove any sensitive information like passwords or URLs.
-      render: yaml
-    validations:
-      required: true
-  - type: textarea
-    id: frigatelogs
-    attributes:
-      label: Relevant Frigate log output
-      description: Please copy and paste any relevant Frigate log output. Include logs before and after your exact error when possible. This will be automatically formatted into code, so no need for backticks.
-      render: shell
-    validations:
-      required: true
-  - type: textarea
-    id: go2rtclogs
-    attributes:
-      label: Relevant go2rtc log output (if applicable)
-      description: If your issue involves cameras, streams, or playback, please include go2rtc logs. Logs can be viewed via the Frigate UI, Docker, or the go2rtc dashboard. This will be automatically formatted into code, so no need for backticks.
-      render: shell
-  - type: dropdown
-    id: install-method
-    attributes:
-      label: Install method
-      options:
-        - Home Assistant Add-on
-        - Docker Compose
-        - Docker CLI
-        - Proxmox via Docker
-        - Proxmox via TTeck Script
-        - Windows WSL2
-    validations:
-      required: true
-  - type: textarea
-    id: docker
-    attributes:
-      label: docker-compose file or Docker CLI command
-      description: This will be automatically formatted into code, so no need for backticks. Include relevant environment variables and device mappings.
-      render: yaml
-    validations:
-      required: true
-  - type: dropdown
-    id: os
-    attributes:
-      label: Operating system
-      options:
-        - Home Assistant OS
-        - Debian
-        - Ubuntu
-        - Other Linux
-        - Proxmox
-        - UNRAID
-        - Windows
-        - Other
-    validations:
-      required: true
-  - type: input
-    id: hardware
-    attributes:
-      label: CPU / GPU / Hardware
-      description: Provide details about your hardware (e.g., Intel i5-9400, NVIDIA RTX 3060, Raspberry Pi 4, etc)
-      placeholder: "Intel i7-10700, NVIDIA GTX 1660"
-  - type: textarea
-    id: screenshots
-    attributes:
-      label: Screenshots
-      description: Screenshots of the issue, System metrics pages, or any relevant UI. Drag and drop or paste images directly.
-  - type: textarea
-    id: steps-to-reproduce
-    attributes:
-      label: Steps to reproduce
-      description: If applicable, provide detailed steps to reproduce the issue
-      placeholder: |
-        1. Go to '...'
-        2. Click on '...'
-        3. See error
-  - type: textarea
-    id: other
-    attributes:
-      label: Any other information that may be helpful
-      description: Additional context, related issues, when the problem started appearing, etc.

.github/DISCUSSION_TEMPLATE/report-a-bug.yml

@@ -6,8 +6,6 @@ body:
       value: |
         Use this form to submit a reproducible bug in Frigate or Frigate's UI.
 
-        **⚠️ If you are running a beta version (0.17.0-beta or similar), please use the [Beta Support template](https://github.com/blakeblackshear/frigate/discussions/new?category=beta-support) instead.**
-
         Before submitting your bug report, please ask the AI with the "Ask AI" button on the [official documentation site][ai] about your issue, [search the discussions][discussions], look at recent open and closed [pull requests][prs], read the [official Frigate documentation][docs], and read the [Frigate FAQ][faq] pinned at the Discussion page to see if your bug has already been fixed by the developers or reported by the community.
 
         **If you are unsure if your issue is actually a bug or not, please submit a support request first.**

.github/copilot-instructions.md

@@ -1,2 +0,0 @@
-Never write strings in the frontend directly, always write to and reference the relevant translations file.
-Always conform new and refactored code to the existing coding style in the project.

.github/workflows/ci.yml

@@ -15,7 +15,7 @@ concurrency:
   cancel-in-progress: true
 
 env:
-  PYTHON_VERSION: 3.11
+  PYTHON_VERSION: 3.9
 
 jobs:
   amd64_build:
@@ -23,7 +23,7 @@ jobs:
     name: AMD64 Build
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx
@@ -47,7 +47,7 @@ jobs:
     name: ARM Build
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx
@@ -82,7 +82,7 @@ jobs:
     name: Jetson Jetpack 6
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx
@@ -113,7 +113,7 @@ jobs:
       - amd64_build
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx
@@ -136,6 +136,7 @@ jobs:
             *.cache-to=type=registry,ref=${{ steps.setup.outputs.cache-name }}-tensorrt,mode=max
       - name: AMD/ROCm general build
         env:
+          AMDGPU: gfx
           HSA_OVERRIDE: 0
         uses: docker/bake-action@v6
         with:
@@ -154,7 +155,7 @@ jobs:
       - arm64_build
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx
@@ -179,7 +180,7 @@ jobs:
       - arm64_build
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up QEMU and Buildx

.github/workflows/pull_request.yml

@@ -16,7 +16,7 @@ jobs:
     name: Web - Lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           persist-credentials: false
       - uses: actions/setup-node@master
@@ -32,7 +32,7 @@ jobs:
     name: Web - Test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           persist-credentials: false
       - uses: actions/setup-node@master
@@ -52,7 +52,7 @@ jobs:
     name: Python Checks
     steps:
       - name: Check out the repository
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - name: Set up Python ${{ env.DEFAULT_PYTHON }}
@@ -75,7 +75,7 @@ jobs:
     name: Python Tests
     steps:
       - name: Check out code
-        uses: actions/checkout@v6
+        uses: actions/checkout@v5
         with:
           persist-credentials: false
       - uses: actions/setup-node@master

.github/workflows/release.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v5
         with:
           persist-credentials: false
       - id: lowercaseRepo

README_CN.md

@@ -1,31 +1,28 @@
 <p align="center">
-  <img align="center" alt="logo" src="docs/static/img/branding/frigate.png">
+  <img align="center" alt="logo" src="docs/static/img/frigate.png">
 </p>
 
-# Frigate NVR™ - 一个具有实时目标检测的本地 NVR
+# Frigate - 一个具有实时目标检测的本地NVR
 
-[English](https://github.com/blakeblackshear/frigate) | \[简体中文\]
-
-[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
+[English](https://github.com/blakeblackshear/frigate) | \[简体中文\] 
 
 <a href="https://hosted.weblate.org/engage/frigate-nvr/-/zh_Hans/">
 <img src="https://hosted.weblate.org/widget/frigate-nvr/-/zh_Hans/svg-badge.svg" alt="翻译状态" />
 </a>
 
-一个完整的本地网络视频录像机（NVR），专为[Home Assistant](https://www.home-assistant.io)设计，具备 AI 目标/物体检测功能。使用 OpenCV 和 TensorFlow 在本地为 IP 摄像头执行实时物体检测。
+一个完整的本地网络视频录像机（NVR），专为[Home Assistant](https://www.home-assistant.io)设计，具备AI物体检测功能。使用OpenCV和TensorFlow在本地为IP摄像头执行实时物体检测。
 
-强烈推荐使用 GPU 或者 AI 加速器（例如[Google Coral 加速器](https://coral.ai/products/) 或者 [Hailo](https://hailo.ai/)等）。它们的运行效率远远高于现在的顶级 CPU，并且功耗也极低。
-
-- 通过[自定义组件](https://github.com/blakeblackshear/frigate-hass-integration)与 Home Assistant 紧密集成
-- 设计上通过仅在必要时和必要地点寻找目标，最大限度地减少资源使用并最大化性能
+强烈推荐使用GPU或者AI加速器（例如[Google Coral加速器](https://coral.ai/products/) 或者 [Hailo](https://hailo.ai/)）。它们的性能甚至超过目前的顶级CPU，并且可以以极低的耗电实现更优的性能。
+- 通过[自定义组件](https://github.com/blakeblackshear/frigate-hass-integration)与Home Assistant紧密集成
+- 设计上通过仅在必要时和必要地点寻找物体，最大限度地减少资源使用并最大化性能
 - 大量利用多进程处理，强调实时性而非处理每一帧
-- 使用非常低开销的画面变动检测（也叫运动检测）来确定运行目标检测的位置
-- 使用 TensorFlow 进行目标检测，并运行在单独的进程中以达到最大 FPS
-- 通过 MQTT 进行通信，便于集成到其他系统中
+- 使用非常低开销的运动检测来确定运行物体检测的位置
+- 使用TensorFlow进行物体检测，运行在单独的进程中以达到最大FPS
+- 通过MQTT进行通信，便于集成到其他系统中
 - 根据检测到的物体设置保留时间进行视频录制
-- 24/7 全天候录制
-- 通过 RTSP 重新流传输以减少摄像头的连接数
-- 支持 WebRTC 和 MSE，实现低延迟的实时观看
+- 24/7全天候录制
+- 通过RTSP重新流传输以减少摄像头的连接数
+- 支持WebRTC和MSE，实现低延迟的实时观看
 
 ## 社区中文翻译文档
 
@@ -35,55 +32,39 @@
 
 如果您想通过捐赠支持开发，请使用 [Github Sponsors](https://github.com/sponsors/blakeblackshear)。
 
-## 协议
-
-本项目采用 **MIT 许可证**授权。
-**代码部分**：本代码库中的源代码、配置文件和文档均遵循 [MIT 许可证](LICENSE)。您可以自由使用、修改和分发这些代码，但必须保留原始版权声明。
-
-**商标部分**：“Frigate”名称、“Frigate NVR”品牌以及 Frigate 的 Logo 为 **Frigate LLC 的商标**，**不在** MIT 许可证覆盖范围内。
-有关品牌资产的规范使用详情，请参阅我们的[《商标政策》](TRADEMARK.md)。
-
 ## 截图
 
 ### 实时监控面板
-
 <div>
 <img width="800" alt="实时监控面板" src="https://github.com/blakeblackshear/frigate/assets/569905/5e713cb9-9db5-41dc-947a-6937c3bc376e">
 </div>
 
 ### 简单的核查工作流程
-
 <div>
 <img width="800" alt="简单的审查工作流程" src="https://github.com/blakeblackshear/frigate/assets/569905/6fed96e8-3b18-40e5-9ddc-31e6f3c9f2ff">
 </div>
 
 ### 多摄像头可按时间轴查看
-
 <div>
 <img width="800" alt="多摄像头可按时间轴查看" src="https://github.com/blakeblackshear/frigate/assets/569905/d6788a15-0eeb-4427-a8d4-80b93cae3d74">
 </div>
 
 ### 内置遮罩和区域编辑器
-
 <div>
 <img width="800" alt="内置遮罩和区域编辑器" src="https://github.com/blakeblackshear/frigate/assets/569905/d7885fc3-bfe6-452f-b7d0-d957cb3e31f5">
 </div>
 
-## 翻译
 
+## 翻译
 我们使用 [Weblate](https://hosted.weblate.org/projects/frigate-nvr/) 平台提供翻译支持，欢迎参与进来一起完善。
 
-## 非官方中文讨论社区
 
-欢迎加入中文讨论 QQ 群：[1043861059](https://qm.qq.com/q/7vQKsTmSz)
+## 非官方中文讨论社区
+欢迎加入中文讨论QQ群：[1043861059](https://qm.qq.com/q/7vQKsTmSz)
 
 Bilibili：https://space.bilibili.com/3546894915602564
 
-## 中文社区赞助商
 
+## 中文社区赞助商
 [![EdgeOne](https://edgeone.ai/media/34fe3a45-492d-4ea4-ae5d-ea1087ca7b4b.png)](https://edgeone.ai/zh?from=github)
 本项目 CDN 加速及安全防护由 Tencent EdgeOne 赞助
-
----
-
-**Copyright © 2025 Frigate LLC.**

docker/main/requirements-wheels.txt

@@ -21,7 +21,7 @@ onvif-zeep-async == 4.0.*
 paho-mqtt == 2.1.*
 pandas == 2.2.*
 peewee == 3.17.*
-peewee_migrate == 1.14.*
+peewee_migrate == 1.13.*
 psutil == 7.1.*
 pydantic == 2.10.*
 git+https://github.com/fbcotter/py3nvml#egg=py3nvml
@@ -81,5 +81,3 @@ librosa==0.11.*
 soundfile==0.13.*
 # DeGirum detector
 degirum == 0.16.*
-# Memory profiling
-memray == 1.15.*

docker/rocm/Dockerfile

@@ -3,6 +3,7 @@
 # https://askubuntu.com/questions/972516/debian-frontend-environment-variable
 ARG DEBIAN_FRONTEND=noninteractive
 ARG ROCM=1
+ARG AMDGPU=gfx900
 ARG HSA_OVERRIDE_GFX_VERSION
 ARG HSA_OVERRIDE
 
@@ -10,10 +11,11 @@ ARG HSA_OVERRIDE
 FROM wget AS rocm
 
 ARG ROCM
+ARG AMDGPU
 
 RUN apt update -qq && \
     apt install -y wget gpg && \
-    wget -O rocm.deb https://repo.radeon.com/amdgpu-install/7.1.1/ubuntu/jammy/amdgpu-install_7.1.1.70101-1_all.deb && \
+    wget -O rocm.deb https://repo.radeon.com/amdgpu-install/7.1/ubuntu/jammy/amdgpu-install_7.1.70100-1_all.deb && \
     apt install -y ./rocm.deb && \
     apt update && \
     apt install -qq -y rocm
@@ -34,10 +36,7 @@ FROM deps AS deps-prelim
 COPY docker/rocm/debian-backports.sources /etc/apt/sources.list.d/debian-backports.sources
 RUN apt-get update && \
     apt-get install -y libnuma1 && \
-    apt-get install -qq -y -t bookworm-backports mesa-va-drivers mesa-vulkan-drivers && \
-    # Install C++ standard library headers for HIPRTC kernel compilation fallback
-    apt-get install -qq -y libstdc++-12-dev && \
-    rm -rf /var/lib/apt/lists/*
+    apt-get install -qq -y -t bookworm-backports mesa-va-drivers mesa-vulkan-drivers
 
 WORKDIR /opt/frigate
 COPY --from=rootfs / /
@@ -55,14 +54,12 @@ RUN pip3 uninstall -y onnxruntime \
 FROM scratch AS rocm-dist
 
 ARG ROCM
+ARG AMDGPU
 
 COPY --from=rocm /opt/rocm-$ROCM/bin/rocminfo /opt/rocm-$ROCM/bin/migraphx-driver /opt/rocm-$ROCM/bin/
-# Copy MIOpen database files for gfx10xx and gfx11xx only (RDNA2/RDNA3)
-COPY --from=rocm /opt/rocm-$ROCM/share/miopen/db/*gfx10* /opt/rocm-$ROCM/share/miopen/db/
-COPY --from=rocm /opt/rocm-$ROCM/share/miopen/db/*gfx11* /opt/rocm-$ROCM/share/miopen/db/
-# Copy rocBLAS library files for gfx10xx and gfx11xx only
-COPY --from=rocm /opt/rocm-$ROCM/lib/rocblas/library/*gfx10* /opt/rocm-$ROCM/lib/rocblas/library/
-COPY --from=rocm /opt/rocm-$ROCM/lib/rocblas/library/*gfx11* /opt/rocm-$ROCM/lib/rocblas/library/
+COPY --from=rocm /opt/rocm-$ROCM/share/miopen/db/*$AMDGPU* /opt/rocm-$ROCM/share/miopen/db/
+COPY --from=rocm /opt/rocm-$ROCM/share/miopen/db/*gfx908* /opt/rocm-$ROCM/share/miopen/db/
+COPY --from=rocm /opt/rocm-$ROCM/lib/rocblas/library/*$AMDGPU* /opt/rocm-$ROCM/lib/rocblas/library/
 COPY --from=rocm /opt/rocm-dist/ /
 
 #######################################################################

docker/rocm/rocm.hcl

@@ -1,5 +1,8 @@
+variable "AMDGPU" {
+  default = "gfx900"
+}
 variable "ROCM" {
-  default = "7.1.1"
+  default = "7.1"
 }
 variable "HSA_OVERRIDE_GFX_VERSION" {
   default = ""
@@ -35,6 +38,7 @@ target rocm {
   }
   platforms = ["linux/amd64"]
   args = {
+    AMDGPU = AMDGPU,
     ROCM = ROCM,
     HSA_OVERRIDE_GFX_VERSION = HSA_OVERRIDE_GFX_VERSION,
     HSA_OVERRIDE = HSA_OVERRIDE

docker/rocm/rocm.mk

@@ -1,15 +1,53 @@
 BOARDS += rocm
 
+# AMD/ROCm is chunky so we build couple of smaller images for specific chipsets
+ROCM_CHIPSETS:=gfx900:9.0.0 gfx1030:10.3.0 gfx1100:11.0.0
+
 local-rocm: version
+	$(foreach chipset,$(ROCM_CHIPSETS), \
+		AMDGPU=$(word 1,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE_GFX_VERSION=$(word 2,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE=1 \
+		docker buildx bake --file=docker/rocm/rocm.hcl rocm \
+			--set rocm.tags=frigate:latest-rocm-$(word 1,$(subst :, ,$(chipset))) \
+			--load \
+	&&) true
+	
+	unset HSA_OVERRIDE_GFX_VERSION && \
+	HSA_OVERRIDE=0 \
+	AMDGPU=gfx \
 	docker buildx bake --file=docker/rocm/rocm.hcl rocm \
 		--set rocm.tags=frigate:latest-rocm \
 		--load
 
 build-rocm: version
+	$(foreach chipset,$(ROCM_CHIPSETS), \
+		AMDGPU=$(word 1,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE_GFX_VERSION=$(word 2,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE=1 \
+		docker buildx bake --file=docker/rocm/rocm.hcl rocm \
+			--set rocm.tags=$(IMAGE_REPO):${GITHUB_REF_NAME}-$(COMMIT_HASH)-rocm-$(chipset) \
+	&&) true
+
+	unset HSA_OVERRIDE_GFX_VERSION && \
+	HSA_OVERRIDE=0 \
+	AMDGPU=gfx \
 	docker buildx bake --file=docker/rocm/rocm.hcl rocm \
 		--set rocm.tags=$(IMAGE_REPO):${GITHUB_REF_NAME}-$(COMMIT_HASH)-rocm
 
 push-rocm: build-rocm
+	$(foreach chipset,$(ROCM_CHIPSETS), \
+		AMDGPU=$(word 1,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE_GFX_VERSION=$(word 2,$(subst :, ,$(chipset))) \
+		HSA_OVERRIDE=1 \
+		docker buildx bake --file=docker/rocm/rocm.hcl rocm \
+			--set rocm.tags=$(IMAGE_REPO):${GITHUB_REF_NAME}-$(COMMIT_HASH)-rocm-$(chipset) \
+			--push \
+	&&) true
+
+	unset HSA_OVERRIDE_GFX_VERSION && \
+	HSA_OVERRIDE=0 \
+	AMDGPU=gfx \
 	docker buildx bake --file=docker/rocm/rocm.hcl rocm \
 		--set rocm.tags=$(IMAGE_REPO):${GITHUB_REF_NAME}-$(COMMIT_HASH)-rocm \
 		--push

docs/docs/configuration/audio_detectors.md

@@ -75,13 +75,7 @@ audio:
 
 ### Audio Transcription
 
-Frigate supports fully local audio transcription using either `sherpa-onnx` or OpenAI’s open-source Whisper models via `faster-whisper`. The goal of this feature is to support Semantic Search for `speech` audio events. Frigate is not intended to act as a continuous, fully-automatic speech transcription service — automatically transcribing all speech (or queuing many audio events for transcription) requires substantial CPU (or GPU) resources and is impractical on most systems. For this reason, transcriptions for events are initiated manually from the UI or the API rather than being run continuously in the background.
-
-Transcription accuracy also depends heavily on the quality of your camera's microphone and recording conditions. Many cameras use inexpensive microphones, and distance to the speaker, low audio bitrate, or background noise can significantly reduce transcription quality. If you need higher accuracy, more robust long-running queues, or large-scale automatic transcription, consider using the HTTP API in combination with an automation platform and a cloud transcription service.
-
-#### Configuration
-
-To enable transcription, enable it in your config. Note that audio detection must also be enabled as described above in order to use audio transcription features.
+Frigate supports fully local audio transcription using either `sherpa-onnx` or OpenAI’s open-source Whisper models via `faster-whisper`. To enable transcription, enable it in your config. Note that audio detection must also be enabled as described above in order to use audio transcription features.
 
 ```yaml
 audio_transcription:
@@ -157,21 +151,3 @@ Only one `speech` event may be transcribed at a time. Frigate does not automatic
 :::
 
 Recorded `speech` events will always use a `whisper` model, regardless of the `model_size` config setting. Without a supported Nvidia GPU, generating transcriptions for longer `speech` events may take a fair amount of time, so be patient.
-
-#### FAQ
-
-1. Why doesn't Frigate automatically transcribe all `speech` events?
-
-   Frigate does not implement a queue mechanism for speech transcription, and adding one is not trivial. A proper queue would need backpressure, prioritization, memory/disk buffering, retry logic, crash recovery, and safeguards to prevent unbounded growth when events outpace processing. That’s a significant amount of complexity for a feature that, in most real-world environments, would mostly just churn through low-value noise.
-
-   Because transcription is **serialized (one event at a time)** and speech events can be generated far faster than they can be processed, an auto-transcribe toggle would very quickly create an ever-growing backlog and degrade core functionality. For the amount of engineering and risk involved, it adds **very little practical value** for the majority of deployments, which are often on low-powered, edge hardware.
-
-   If you hear speech that’s actually important and worth saving/indexing for the future, **just press the transcribe button in Explore** on that specific `speech` event - that keeps things explicit, reliable, and under your control.
-
-   Other options are being considered for future versions of Frigate to add transcription options that support external `whisper` Docker containers. A single transcription service could then be shared by Frigate and other applications (for example, Home Assistant Voice), and run on more powerful machines when available.
-
-2. Why don't you save live transcription text and use that for `speech` events?
-
-   There’s no guarantee that a `speech` event is even created from the exact audio that went through the transcription model. Live transcription and `speech` event creation are **separate, asynchronous processes**. Even when both are correctly configured, trying to align the **precise start and end time of a speech event** with whatever audio the model happened to be processing at that moment is unreliable.
-
-   Automatically persisting that data would often result in **misaligned, partial, or irrelevant transcripts**, while still incurring all of the CPU, storage, and privacy costs of transcription. That’s why Frigate treats transcription as an **explicit, user-initiated action** rather than an automatic side-effect of every `speech` event.

docs/docs/configuration/custom_classification/state_classification.md

@@ -69,6 +69,4 @@ Once all images are assigned, training will begin automatically.
 ### Improving the Model
 
 - **Problem framing**: Keep classes visually distinct and state-focused (e.g., `open`, `closed`, `unknown`). Avoid combining object identity with state in a single model unless necessary.
-- **Data collection**: Use the model's Recent Classifications tab to gather balanced examples across times of day and weather.
-- **When to train**: Focus on cases where the model is entirely incorrect or flips between states when it should not. There's no need to train additional images when the model is already working consistently.
-- **Selecting training images**: Images scoring below 100% due to new conditions (e.g., first snow of the year, seasonal changes) or variations (e.g., objects temporarily in view, insects at night) are good candidates for training, as they represent scenarios different from the default state. Training these lower-scoring images that differ from existing training data helps prevent overfitting. Avoid training large quantities of images that look very similar, especially if they already score 100% as this can lead to overfitting.
+- **Data collection**: Use the model’s Recent Classifications tab to gather balanced examples across times of day and weather.

docs/docs/configuration/genai/review_summaries.md

@@ -111,9 +111,3 @@ review:
 ## Review Reports
 
 Along with individual review item summaries, Generative AI provides the ability to request a report of a given time period. For example, you can get a daily report while on a vacation of any suspicious activity or other concerns that may require review.
-
-### Requesting Reports Programmatically
-
-Review reports can be requested via the [API](/integrations/api#review-summarization) by sending a POST request to `/api/review/summarize/start/{start_ts}/end/{end_ts}` with Unix timestamps.
-
-For Home Assistant users, there is a built-in service (`frigate.review_summarize`) that makes it easy to request review reports as part of automations or scripts. This allows you to automatically generate daily summaries, vacation reports, or custom time period reports based on your specific needs.

docs/docs/configuration/license_plate_recognition.md

@@ -107,23 +107,23 @@ Fine-tune the LPR feature using these optional parameters at the global level of
 
 ### Normalization Rules
 
-- **`replace_rules`**: List of regex replacement rules to normalize detected plates. These rules are applied sequentially and are applied _before_ the `format` regex, if specified. Each rule must have a `pattern` (which can be a string or a regex) and `replacement` (a string, which also supports [backrefs](https://docs.python.org/3/library/re.html#re.sub) like `\1`). These rules are useful for dealing with common OCR issues like noise characters, separators, or confusions (e.g., 'O'→'0').
+- **`replace_rules`**: List of regex replacement rules to normalize detected plates. These rules are applied sequentially. Each rule must have a `pattern` (which can be a string or a regex, prepended by `r`) and `replacement` (a string, which also supports [backrefs](https://docs.python.org/3/library/re.html#re.sub) like `\1`). These rules are useful for dealing with common OCR issues like noise characters, separators, or confusions (e.g., 'O'→'0').
 
 These rules must be defined at the global level of your `lpr` config.
 
 ```yaml
 lpr:
   replace_rules:
-    - pattern: "[%#*?]" # Remove noise symbols
+    - pattern: r'[%#*?]' # Remove noise symbols
       replacement: ""
-    - pattern: "[= ]" # Normalize = or space to dash
+    - pattern: r'[= ]' # Normalize = or space to dash
       replacement: "-"
     - pattern: "O" # Swap 'O' to '0' (common OCR error)
       replacement: "0"
-    - pattern: "I" # Swap 'I' to '1'
+    - pattern: r'I' # Swap 'I' to '1'
       replacement: "1"
-    - pattern: '(\w{3})(\w{3})' # Split 6 chars into groups (e.g., ABC123 → ABC-123) - use single quotes to preserve backslashes
-      replacement: '\1-\2'
+    - pattern: r'(\w{3})(\w{3})' # Split 6 chars into groups (e.g., ABC123 → ABC-123)
+      replacement: r'\1-\2'
 ```
 
 - Rules fire in order: In the example above: clean noise first, then separators, then swaps, then splits.
@@ -374,19 +374,9 @@ Use `match_distance` to allow small character mismatches. Alternatively, define
 
 Start with ["Why isn't my license plate being detected and recognized?"](#why-isnt-my-license-plate-being-detected-and-recognized). If you are still having issues, work through these steps.
 
-1. Start with a simplified LPR config.
+1. Enable debug logs to see exactly what Frigate is doing.
 
-   - Remove or comment out everything in your LPR config, including `min_area`, `min_plate_length`, `format`, `known_plates`, or `enhancement` values so that the only values left are `enabled` and `debug_save_plates`. This will run LPR with Frigate's default values.
-
-     ```yaml
-     lpr:
-       enabled: true
-       debug_save_plates: true
-     ```
-
-2. Enable debug logs to see exactly what Frigate is doing.
-
-   - Enable debug logs for LPR by adding `frigate.data_processing.common.license_plate: debug` to your `logger` configuration. These logs are _very_ verbose, so only keep this enabled when necessary. Restart Frigate after this change.
+   - Enable debug logs for LPR by adding `frigate.data_processing.common.license_plate: debug` to your `logger` configuration. These logs are _very_ verbose, so only keep this enabled when necessary.
 
      ```yaml
      logger:
@@ -395,7 +385,7 @@ Start with ["Why isn't my license plate being detected and recognized?"](#why-is
          frigate.data_processing.common.license_plate: debug
      ```
 
-3. Ensure your plates are being _detected_.
+2. Ensure your plates are being _detected_.
 
    If you are using a Frigate+ or `license_plate` detecting model:
 
@@ -408,7 +398,7 @@ Start with ["Why isn't my license plate being detected and recognized?"](#why-is
    - Watch the debug logs for messages from the YOLOv9 plate detector.
    - You may need to adjust your `detection_threshold` if your plates are not being detected.
 
-4. Ensure the characters on detected plates are being _recognized_.
+3. Ensure the characters on detected plates are being _recognized_.
 
    - Enable `debug_save_plates` to save images of detected text on plates to the clips directory (`/media/frigate/clips/lpr`). Ensure these images are readable and the text is clear.
    - Watch the debug view to see plates recognized in real-time. For non-dedicated LPR cameras, the `car` or `motorcycle` label will change to the recognized plate when LPR is enabled and working.

docs/docs/configuration/live.md

@@ -178,8 +178,6 @@ To use the Reolink Doorbell with two way talk, you should use the [recommended R
 
 As a starting point to check compatibility for your camera, view the list of cameras supported for two-way talk on the [go2rtc repository](https://github.com/AlexxIT/go2rtc?tab=readme-ov-file#two-way-audio). For cameras in the category `ONVIF Profile T`, you can use the [ONVIF Conformant Products Database](https://www.onvif.org/conformant-products/)'s FeatureList to check for the presence of `AudioOutput`. A camera that supports `ONVIF Profile T` _usually_ supports this, but due to inconsistent support, a camera that explicitly lists this feature may still not work. If no entry for your camera exists on the database, it is recommended not to buy it or to consult with the manufacturer's support on the feature availability.
 
-To prevent go2rtc from blocking other applications from accessing your camera's two-way audio, you must configure your stream with `#backchannel=0`. See [preventing go2rtc from blocking two-way audio](/configuration/restream#two-way-talk-restream) in the restream documentation.
-
 ### Streaming options on camera group dashboards
 
 Frigate provides a dialog in the Camera Group Edit pane with several options for streaming on a camera group's dashboard. These settings are _per device_ and are saved in your device's local storage.

docs/docs/configuration/masks.md

@@ -28,6 +28,7 @@ To create a poly mask:
 5. Click the plus icon under the type of mask or zone you would like to create
 6. Click on the camera's latest image to create the points for a masked area. Click the first point again to close the polygon.
 7. When you've finished creating your mask, press Save.
+8. Restart Frigate to apply your changes.
 
 Your config file will be updated with the relative coordinates of the mask/zone:
 

docs/docs/configuration/object_detectors.md

@@ -13,7 +13,7 @@ Frigate supports multiple different detectors that work on different types of ha
 
 **Most Hardware**
 
-- [Coral EdgeTPU](#edge-tpu-detector): The Google Coral EdgeTPU is available in USB, Mini PCIe, and m.2 formats allowing for a wide range of compatibility with devices.
+- [Coral EdgeTPU](#edge-tpu-detector): The Google Coral EdgeTPU is available in USB and m.2 format allowing for a wide range of compatibility with devices.
 - [Hailo](#hailo-8): The Hailo8 and Hailo8L AI Acceleration module is available in m.2 format with a HAT for RPi devices, offering a wide range of compatibility with devices.
 - <CommunityBadge /> [MemryX](#memryx-mx3): The MX3 Acceleration module is available in m.2 format, offering broad compatibility across various platforms.
 - <CommunityBadge /> [DeGirum](#degirum): Service for using hardware devices in the cloud or locally. Hardware and models provided on the cloud on [their website](https://hub.degirum.com).
@@ -69,10 +69,12 @@ Frigate provides the following builtin detector types: `cpu`, `edgetpu`, `hailo8
 
 ## Edge TPU Detector
 
-The Edge TPU detector type runs TensorFlow Lite models utilizing the Google Coral delegate for hardware acceleration. To configure an Edge TPU detector, set the `"type"` attribute to `"edgetpu"`.
+The Edge TPU detector type runs a TensorFlow Lite model utilizing the Google Coral delegate for hardware acceleration. To configure an Edge TPU detector, set the `"type"` attribute to `"edgetpu"`.
 
 The Edge TPU device can be specified using the `"device"` attribute according to the [Documentation for the TensorFlow Lite Python API](https://coral.ai/docs/edgetpu/multiple-edgetpu/#using-the-tensorflow-lite-python-api). If not set, the delegate will use the first device it finds.
 
+A TensorFlow Lite model is provided in the container at `/edgetpu_model.tflite` and is used by this detector type by default. To provide your own model, bind mount the file into the container and provide the path with `model.path`.
+
 :::tip
 
 See [common Edge TPU troubleshooting steps](/troubleshooting/edgetpu) if the Edge TPU is not detected.
@@ -144,44 +146,6 @@ detectors:
     device: pci
 ```
 
-### EdgeTPU Supported Models
-
-| Model                                 | Notes                                       |
-| ------------------------------------- | ------------------------------------------- |
-| [MobileNet v2](#ssdlite-mobilenet-v2) | Default model                               |
-| [YOLOv9](#yolo-v9)                    | More accurate but slower than default model |
-
-#### SSDLite MobileNet v2
-
-A TensorFlow Lite model is provided in the container at `/edgetpu_model.tflite` and is used by this detector type by default. To provide your own model, bind mount the file into the container and provide the path with `model.path`.
-
-#### YOLO v9
-
-[YOLOv9](https://github.com/dbro/frigate-detector-edgetpu-yolo9/releases/download/v1.0/yolov9-s-relu6-best_320_int8_edgetpu.tflite) models that are compiled for Tensorflow Lite and properly quantized are supported, but not included by default. To provide your own model, bind mount the file into the container and provide the path with `model.path`. Note that the model may require a custom label file (eg. [use this 17 label file](https://raw.githubusercontent.com/dbro/frigate-detector-edgetpu-yolo9/refs/heads/main/labels-coco17.txt) for the model linked above.)
-
-<details>
-  <summary>YOLOv9 Setup & Config</summary>
-
-After placing the downloaded files for the tflite model and labels in your config folder, you can use the following configuration:
-
-```yaml
-detectors:
-  coral:
-    type: edgetpu
-    device: usb
-
-model:
-  model_type: yolo-generic
-  width: 320 # <--- should match the imgsize of the model, typically 320
-  height: 320 # <--- should match the imgsize of the model, typically 320
-  path: /config/model_cache/yolov9-s-relu6-best_320_int8_edgetpu.tflite
-  labelmap_path: /labelmap/labels-coco-17.txt
-```
-
-Note that the labelmap uses a subset of the complete COCO label set that has only 17 objects.
-
-</details>
-
 ---
 
 ## Hailo-8
@@ -400,7 +364,7 @@ The YOLO detector has been designed to support YOLOv3, YOLOv4, YOLOv7, and YOLOv
 
 :::warning
 
-If you are using a Frigate+ model, you should not define any of the below `model` parameters in your config except for `path`. See [the Frigate+ model docs](/plus/first_model#step-3-set-your-model-id-in-the-config) for more information on setting up your model.
+If you are using a Frigate+ YOLOv9 model, you should not define any of the below `model` parameters in your config except for `path`. See [the Frigate+ model docs](/plus/first_model#step-3-set-your-model-id-in-the-config) for more information on setting up your model.
 
 :::
 
@@ -740,7 +704,7 @@ The YOLO detector has been designed to support YOLOv3, YOLOv4, YOLOv7, and YOLOv
 
 :::warning
 
-If you are using a Frigate+ model, you should not define any of the below `model` parameters in your config except for `path`. See [the Frigate+ model docs](/plus/first_model#step-3-set-your-model-id-in-the-config) for more information on setting up your model.
+If you are using a Frigate+ YOLOv9 model, you should not define any of the below `model` parameters in your config except for `path`. See [the Frigate+ model docs](/plus/first_model#step-3-set-your-model-id-in-the-config) for more information on setting up your model.
 
 :::
 

docs/docs/configuration/reference.md

@@ -123,7 +123,7 @@ auth:
   # Optional: Refresh time in seconds (default: shown below)
   # When the session is going to expire in less time than this setting,
   # it will be refreshed back to the session_length.
-  refresh_time: 1800 # 30 minutes
+  refresh_time: 43200 # 12 hours
   # Optional: Rate limiting for login failures to help prevent brute force
   # login attacks (default: shown below)
   # See the docs for more information on valid values
@@ -710,44 +710,6 @@ audio_transcription:
   # List of language codes: https://github.com/openai/whisper/blob/main/whisper/tokenizer.py#L10
   language: en
 
-# Optional: Configuration for classification models
-classification:
-  # Optional: Configuration for bird classification
-  bird:
-    # Optional: Enable bird classification (default: shown below)
-    enabled: False
-    # Optional: Minimum classification score required to be considered a match (default: shown below)
-    threshold: 0.9
-  custom:
-    # Required: name of the classification model
-    model_name:
-      # Optional: Enable running the model (default: shown below)
-      enabled: True
-      # Optional: Name of classification model (default: shown below)
-      name: None
-      # Optional: Classification score threshold to change the state (default: shown below)
-      threshold: 0.8
-      # Optional: Number of classification attempts to save in the recent classifications tab (default: shown below)
-      # NOTE: Defaults to 200 for object classification and 100 for state classification if not specified
-      save_attempts: None
-      # Optional: Object classification configuration
-      object_config:
-        # Required: Object types to classify
-        objects: [dog]
-        # Optional: Type of classification that is applied (default: shown below)
-        classification_type: sub_label
-      # Optional: State classification configuration
-      state_config:
-        # Required: Cameras to run classification on
-        cameras:
-          camera_name:
-            # Required: Crop of image frame on this camera to run classification on
-            crop: [0, 180, 220, 400]
-        # Optional: If classification should be run when motion is detected in the crop (default: shown below)
-        motion: False
-        # Optional: Interval to run classification on in seconds (default: shown below)
-        interval: None
-
 # Optional: Restream configuration
 # Uses https://github.com/AlexxIT/go2rtc (v1.9.10)
 # NOTE: The default go2rtc API port (1984) must be used,
@@ -911,7 +873,7 @@ cameras:
       user: admin
       # Optional: password for login.
       password: admin
-      # Optional: Skip TLS verification and disable digest authentication for the ONVIF server (default: shown below)
+      # Optional: Skip TLS verification from the ONVIF server (default: shown below)
       tls_insecure: False
       # Optional: Ignores time synchronization mismatches between the camera and the server during authentication.
       # Using NTP on both ends is recommended and this should only be set to True in a "safe" environment due to the security risk it represents.
@@ -1002,6 +964,10 @@ ui:
   #    full: 8:15:22 PM Mountain Standard Time
   # (default: shown below).
   time_style: medium
+  # Optional: Ability to manually override the date / time styling to use strftime format
+  # https://www.gnu.org/software/libc/manual/html_node/Formatting-Calendar-Time.html
+  # possible values are shown above (default: not set)
+  strftime_fmt: "%Y/%m/%d %H:%M"
   # Optional: Set the unit system to either "imperial" or "metric" (default: metric)
   # Used in the UI and in MQTT topics
   unit_system: metric

docs/docs/configuration/restream.md

@@ -24,12 +24,11 @@ birdseye:
   restream: True
 ```
 
-:::tip
+:::tip 
 
-To improve connection speed when using Birdseye via restream you can enable a small idle heartbeat by setting `birdseye.idle_heartbeat_fps` to a low value (e.g. `1–2`). This makes Frigate periodically push the last frame even when no motion is detected, reducing initial connection latency.
+To improve connection speed when using Birdseye via restream you can enable a small idle heartbeat by setting `birdseye.idle_heartbeat_fps` to a low value (e.g. `1–2`).  This makes Frigate periodically push the last frame even when no motion is detected, reducing initial connection latency.
 
 :::
-
 ### Securing Restream With Authentication
 
 The go2rtc restream can be secured with RTSP based username / password authentication. Ex:
@@ -160,31 +159,6 @@ go2rtc:
 
 See [this comment](https://github.com/AlexxIT/go2rtc/issues/1217#issuecomment-2242296489) for more information.
 
-## Preventing go2rtc from blocking two-way audio {#two-way-talk-restream}
-
-For cameras that support two-way talk, go2rtc will automatically establish an audio output backchannel when connecting to an RTSP stream. This backchannel blocks access to the camera's audio output for two-way talk functionality, preventing both Frigate and other applications from using it.
-
-To prevent this, you must configure two separate stream instances:
-
-1. One stream instance with `#backchannel=0` for Frigate's viewing, recording, and detection (prevents go2rtc from establishing the blocking backchannel)
-2. A second stream instance without `#backchannel=0` for two-way talk functionality (can be used by Frigate's WebRTC viewer or other applications)
-
-Configuration example:
-
-```yaml
-go2rtc:
-  streams:
-    front_door:
-      - rtsp://user:password@10.0.10.10:554/cam/realmonitor?channel=1&subtype=2#backchannel=0
-    front_door_twoway:
-      - rtsp://user:password@10.0.10.10:554/cam/realmonitor?channel=1&subtype=2
-```
-
-In this configuration:
-
-- `front_door` stream is used by Frigate for viewing, recording, and detection. The `#backchannel=0` parameter prevents go2rtc from establishing the audio output backchannel, so it won't block two-way talk access.
-- `front_door_twoway` stream is used for two-way talk functionality. This stream can be used by Frigate's WebRTC viewer when two-way talk is enabled, or by other applications (like Home Assistant Advanced Camera Card) that need access to the camera's audio output channel.
-
 ## Advanced Restream Configurations
 
 The [exec](https://github.com/AlexxIT/go2rtc/tree/v1.9.10#source-exec) source in go2rtc can be used for custom ffmpeg commands. An example is below:
@@ -195,4 +169,4 @@ NOTE: The output will need to be passed with two curly braces `{{output}}`
 go2rtc:
   streams:
     stream1: exec:ffmpeg -hide_banner -re -stream_loop -1 -i /media/BigBuckBunny.mp4 -c copy -rtsp_transport tcp -f rtsp {{output}}
-```
+```

docs/docs/frigate/hardware.md

@@ -159,7 +159,7 @@ Inference speeds vary greatly depending on the CPU or GPU used, some known examp
 | Intel HD 530   | 15 - 35 ms                 |                                                   |                           |                        | Can only run one detector instance |
 | Intel HD 620   | 15 - 25 ms                 |                                                   | 320: ~ 35 ms              |                        |                                    |
 | Intel HD 630   | ~ 15 ms                    |                                                   | 320: ~ 30 ms              |                        |                                    |
-| Intel UHD 730  | ~ 10 ms                    | t-320: 14ms s-320: 24ms t-640: 34ms s-640: 65ms   | 320: ~ 19 ms 640: ~ 54 ms |                        |                                    |
+| Intel UHD 730  | ~ 10 ms                    |                                                   | 320: ~ 19 ms 640: ~ 54 ms |                        |                                    |
 | Intel UHD 770  | ~ 15 ms                    | t-320: ~ 16 ms s-320: ~ 20 ms s-640: ~ 40 ms      | 320: ~ 20 ms 640: ~ 46 ms |                        |                                    |
 | Intel N100     | ~ 15 ms                    | s-320: 30 ms                                      | 320: ~ 25 ms              |                        | Can only run one detector instance |
 | Intel N150     | ~ 15 ms                    | t-320: 16 ms s-320: 24 ms                         |                           |                        |                                    |

docs/docs/frigate/installation.md

@@ -135,7 +135,6 @@ Finally, configure [hardware object detection](/configuration/object_detectors#h
 ### MemryX MX3
 
 The MemryX MX3 Accelerator is available in the M.2 2280 form factor (like an NVMe SSD), and supports a variety of configurations:
-
 - x86 (Intel/AMD) PCs
 - Raspberry Pi 5
 - Orange Pi 5 Plus/Max
@@ -143,6 +142,7 @@ The MemryX MX3 Accelerator is available in the M.2 2280 form factor (like an NVM
 
 #### Configuration
 
+
 #### Installation
 
 To get started with MX3 hardware setup for your system, refer to the [Hardware Setup Guide](https://developer.memryx.com/get_started/hardware_setup.html).
@@ -156,7 +156,7 @@ Then follow these steps for installing the correct driver/runtime configuration:
 
 #### Setup
 
-To set up Frigate, follow the default installation instructions, for example: `ghcr.io/blakeblackshear/frigate:stable`
+To set up Frigate, follow the default installation instructions, for example:   `ghcr.io/blakeblackshear/frigate:stable`
 
 Next, grant Docker permissions to access your hardware by adding the following lines to your `docker-compose.yml` file:
 
@@ -173,7 +173,7 @@ In your `docker-compose.yml`, also add:
 privileged: true
 
 volumes:
-  - /run/mxa_manager:/run/mxa_manager
+    /run/mxa_manager:/run/mxa_manager
 ```
 
 If you can't use Docker Compose, you can run the container with something similar to this:
@@ -411,7 +411,7 @@ To install make sure you have the [community app plugin here](https://forums.unr
 
 ## Proxmox
 
-[According to Proxmox documentation](https://pve.proxmox.com/pve-docs/pve-admin-guide.html#chapter_pct) it is recommended that you run application containers like Frigate inside a Proxmox QEMU VM. This will give you all the advantages of application containerization, while also providing the benefits that VMs offer, such as strong isolation from the host and the ability to live-migrate, which otherwise isn’t possible with containers. Ensure that ballooning is **disabled**, especially if you are passing through a GPU to the VM.
+[According to Proxmox documentation](https://pve.proxmox.com/pve-docs/pve-admin-guide.html#chapter_pct) it is recommended that you run application containers like Frigate inside a Proxmox QEMU VM. This will give you all the advantages of application containerization, while also providing the benefits that VMs offer, such as strong isolation from the host and the ability to live-migrate, which otherwise isn’t possible with containers.
 
 :::warning
 

docs/docs/guides/configuring_go2rtc.md

@@ -113,8 +113,7 @@ section.
 
 1. If the stream you added to go2rtc is also used by Frigate for the `record` or `detect` role, you can migrate your config to pull from the RTSP restream to reduce the number of connections to your camera as shown [here](/configuration/restream#reduce-connections-to-camera).
 2. You can [set up WebRTC](/configuration/live#webrtc-extra-configuration) if your camera supports two-way talk. Note that WebRTC only supports specific audio formats and may require opening ports on your router.
-3. If your camera supports two-way talk, you must configure your stream with `#backchannel=0` to prevent go2rtc from blocking other applications from accessing the camera's audio output. See [preventing go2rtc from blocking two-way audio](/configuration/restream#two-way-talk-restream) in the restream documentation.
 
 ## Homekit Configuration
 
-To add camera streams to Homekit Frigate must be configured in docker to use `host` networking mode. Once that is done, you can use the go2rtc WebUI (accessed via port 1984, which is disabled by default) to share export a camera to Homekit. Any changes made will automatically be saved to `/config/go2rtc_homekit.yml`.
+To add camera streams to Homekit Frigate must be configured in docker to use `host` networking mode. Once that is done, you can use the go2rtc WebUI (accessed via port 1984, which is disabled by default) to share export a camera to Homekit. Any changes made will automatically be saved to `/config/go2rtc_homekit.yml`.

docs/docs/troubleshooting/memory.md

@@ -1,129 +0,0 @@
----
-id: memory
-title: Memory Troubleshooting
----
-
-Frigate includes built-in memory profiling using [memray](https://bloomberg.github.io/memray/) to help diagnose memory issues. This feature allows you to profile specific Frigate modules to identify memory leaks, excessive allocations, or other memory-related problems.
-
-## Enabling Memory Profiling
-
-Memory profiling is controlled via the `FRIGATE_MEMRAY_MODULES` environment variable. Set it to a comma-separated list of module names you want to profile:
-
-```bash
-export FRIGATE_MEMRAY_MODULES="frigate.review_segment_manager,frigate.capture"
-```
-
-### Module Names
-
-Frigate processes are named using a module-based naming scheme. Common module names include:
-
-- `frigate.review_segment_manager` - Review segment processing
-- `frigate.recording_manager` - Recording management
-- `frigate.capture` - Camera capture processes (all cameras with this module name)
-- `frigate.process` - Camera processing/tracking (all cameras with this module name)
-- `frigate.output` - Output processing
-- `frigate.audio_manager` - Audio processing
-- `frigate.embeddings` - Embeddings processing
-
-You can also specify the full process name (including camera-specific identifiers) if you want to profile a specific camera:
-
-```bash
-export FRIGATE_MEMRAY_MODULES="frigate.capture:front_door"
-```
-
-When you specify a module name (e.g., `frigate.capture`), all processes with that module prefix will be profiled. For example, `frigate.capture` will profile all camera capture processes.
-
-## How It Works
-
-1. **Binary File Creation**: When profiling is enabled, memray creates a binary file (`.bin`) in `/config/memray_reports/` that is updated continuously in real-time as the process runs.
-
-2. **Automatic HTML Generation**: On normal process exit, Frigate automatically:
-
-   - Stops memray tracking
-   - Generates an HTML flamegraph report
-   - Saves it to `/config/memray_reports/<module_name>.html`
-
-3. **Crash Recovery**: If a process crashes (SIGKILL, segfault, etc.), the binary file is preserved with all data up to the crash point. You can manually generate the HTML report from the binary file.
-
-## Viewing Reports
-
-### Automatic Reports
-
-After a process exits normally, you'll find HTML reports in `/config/memray_reports/`. Open these files in a web browser to view interactive flamegraphs showing memory usage patterns.
-
-### Manual Report Generation
-
-If a process crashes or you want to generate a report from an existing binary file, you can manually create the HTML report:
-
-```bash
-memray flamegraph /config/memray_reports/<module_name>.bin
-```
-
-This will generate an HTML file that you can open in your browser.
-
-## Understanding the Reports
-
-Memray flamegraphs show:
-
-- **Memory allocations over time**: See where memory is being allocated in your code
-- **Call stacks**: Understand the full call chain leading to allocations
-- **Memory hotspots**: Identify functions or code paths that allocate the most memory
-- **Memory leaks**: Spot patterns where memory is allocated but not freed
-
-The interactive HTML reports allow you to:
-
-- Zoom into specific time ranges
-- Filter by function names
-- View detailed allocation information
-- Export data for further analysis
-
-## Best Practices
-
-1. **Profile During Issues**: Enable profiling when you're experiencing memory issues, not all the time, as it adds some overhead.
-
-2. **Profile Specific Modules**: Instead of profiling everything, focus on the modules you suspect are causing issues.
-
-3. **Let Processes Run**: Allow processes to run for a meaningful duration to capture representative memory usage patterns.
-
-4. **Check Binary Files**: If HTML reports aren't generated automatically (e.g., after a crash), check for `.bin` files in `/config/memray_reports/` and generate reports manually.
-
-5. **Compare Reports**: Generate reports at different times to compare memory usage patterns and identify trends.
-
-## Troubleshooting
-
-### No Reports Generated
-
-- Check that the environment variable is set correctly
-- Verify the module name matches exactly (case-sensitive)
-- Check logs for memray-related errors
-- Ensure `/config/memray_reports/` directory exists and is writable
-
-### Process Crashed Before Report Generation
-
-- Look for `.bin` files in `/config/memray_reports/`
-- Manually generate HTML reports using: `memray flamegraph <file>.bin`
-- The binary file contains all data up to the crash point
-
-### Reports Show No Data
-
-- Ensure the process ran long enough to generate meaningful data
-- Check that memray is properly installed (included by default in Frigate)
-- Verify the process actually started and ran (check process logs)
-
-## Example Usage
-
-```bash
-# Enable profiling for review and capture modules
-export FRIGATE_MEMRAY_MODULES="frigate.review_segment_manager,frigate.capture"
-
-# Start Frigate
-# ... let it run for a while ...
-
-# Check for reports
-ls -lh /config/memray_reports/
-
-# If a process crashed, manually generate report
-memray flamegraph /config/memray_reports/frigate_capture_front_door.bin
-```
-
-For more information about memray and interpreting reports, see the [official memray documentation](https://bloomberg.github.io/memray/).

docs/package.json

@@ -18,14 +18,14 @@
   },
   "dependencies": {
     "@docusaurus/core": "^3.7.0",
-    "@docusaurus/plugin-content-docs": "^3.7.0",
+    "@docusaurus/plugin-content-docs": "^3.6.3",
     "@docusaurus/preset-classic": "^3.7.0",
-    "@docusaurus/theme-mermaid": "^3.7.0",
+    "@docusaurus/theme-mermaid": "^3.6.3",
     "@inkeep/docusaurus": "^2.0.16",
     "@mdx-js/react": "^3.1.0",
     "clsx": "^2.1.1",
-    "docusaurus-plugin-openapi-docs": "^4.5.1",
-    "docusaurus-theme-openapi-docs": "^4.5.1",
+    "docusaurus-plugin-openapi-docs": "^4.3.1",
+    "docusaurus-theme-openapi-docs": "^4.3.1",
     "prism-react-renderer": "^2.4.1",
     "raw-loader": "^4.0.2",
     "react": "^18.3.1",
@@ -44,9 +44,9 @@
     ]
   },
   "devDependencies": {
-    "@docusaurus/module-type-aliases": "^3.7.0",
-    "@docusaurus/types": "^3.7.0",
-    "@types/react": "^18.3.27"
+    "@docusaurus/module-type-aliases": "^3.4.0",
+    "@docusaurus/types": "^3.4.0",
+    "@types/react": "^18.3.7"
   },
   "engines": {
     "node": ">=18.0"

docs/sidebars.ts

@@ -131,7 +131,6 @@ const sidebars: SidebarsConfig = {
       "troubleshooting/recordings",
       "troubleshooting/gpu",
       "troubleshooting/edgetpu",
-      "troubleshooting/memory",
     ],
     Development: [
       "development/contributing",

docs/src/components/LanguageAlert/styles.module.css

@@ -1,18 +1,13 @@
 .alert {
-  padding: 12px;
-  background: #fff8e6;
-  border-bottom: 1px solid #ffd166;
-  text-align: center;
-  font-size: 15px;
-}
-
-[data-theme="dark"] .alert {
-  background: #3b2f0b;
-  border-bottom: 1px solid #665c22;
-}
-
-.alert a {
-  color: #1890ff;
-  font-weight: 500;
-  margin-left: 6px;
-}
+    padding: 12px;
+    background: #fff8e6;
+    border-bottom: 1px solid #ffd166;
+    text-align: center;
+    font-size: 15px;
+  }
+  
+  .alert a {
+    color: #1890ff;
+    font-weight: 500;
+    margin-left: 6px;
+  }

frigate/api/app.py

@@ -23,7 +23,7 @@ from markupsafe import escape
 from peewee import SQL, fn, operator
 from pydantic import ValidationError
 
-from frigate.api.auth import allow_any_authenticated, allow_public, require_role
+from frigate.api.auth import require_role
 from frigate.api.defs.query.app_query_parameters import AppTimelineHourlyQueryParameters
 from frigate.api.defs.request.app_body import AppConfigSetBody
 from frigate.api.defs.tags import Tags
@@ -56,33 +56,29 @@ logger = logging.getLogger(__name__)
 router = APIRouter(tags=[Tags.app])
 
 
-@router.get(
-    "/", response_class=PlainTextResponse, dependencies=[Depends(allow_public())]
-)
+@router.get("/", response_class=PlainTextResponse)
 def is_healthy():
     return "Frigate is running. Alive and healthy!"
 
 
-@router.get("/config/schema.json", dependencies=[Depends(allow_public())])
+@router.get("/config/schema.json")
 def config_schema(request: Request):
     return Response(
         content=request.app.frigate_config.schema_json(), media_type="application/json"
     )
 
 
-@router.get(
-    "/version", response_class=PlainTextResponse, dependencies=[Depends(allow_public())]
-)
+@router.get("/version", response_class=PlainTextResponse)
 def version():
     return VERSION
 
 
-@router.get("/stats", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/stats")
 def stats(request: Request):
     return JSONResponse(content=request.app.stats_emitter.get_latest_stats())
 
 
-@router.get("/stats/history", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/stats/history")
 def stats_history(request: Request, keys: str = None):
     if keys:
         keys = keys.split(",")
@@ -90,7 +86,7 @@ def stats_history(request: Request, keys: str = None):
     return JSONResponse(content=request.app.stats_emitter.get_stats_history(keys))
 
 
-@router.get("/metrics", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/metrics")
 def metrics(request: Request):
     """Expose Prometheus metrics endpoint and update metrics with latest stats"""
     # Retrieve the latest statistics and update the Prometheus metrics
@@ -107,7 +103,7 @@ def metrics(request: Request):
     return Response(content=content, media_type=content_type)
 
 
-@router.get("/config", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/config")
 def config(request: Request):
     config_obj: FrigateConfig = request.app.frigate_config
     config: dict[str, dict[str, Any]] = config_obj.model_dump(
@@ -213,7 +209,7 @@ def config_raw_paths(request: Request):
     return JSONResponse(content=raw_paths)
 
 
-@router.get("/config/raw", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/config/raw")
 def config_raw():
     config_file = find_config_file()
 
@@ -456,7 +452,7 @@ def config_set(request: Request, body: AppConfigSetBody):
     )
 
 
-@router.get("/vainfo", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/vainfo")
 def vainfo():
     vainfo = vainfo_hwaccel()
     return JSONResponse(
@@ -476,16 +472,12 @@ def vainfo():
     )
 
 
-@router.get("/nvinfo", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/nvinfo")
 def nvinfo():
     return JSONResponse(content=get_nvidia_driver_info())
 
 
-@router.get(
-    "/logs/{service}",
-    tags=[Tags.logs],
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/logs/{service}", tags=[Tags.logs])
 async def logs(
     service: str = Path(enum=["frigate", "nginx", "go2rtc"]),
     download: Optional[str] = None,
@@ -593,7 +585,7 @@ def restart():
     )
 
 
-@router.get("/labels", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/labels")
 def get_labels(camera: str = ""):
     try:
         if camera:
@@ -611,7 +603,7 @@ def get_labels(camera: str = ""):
     return JSONResponse(content=labels)
 
 
-@router.get("/sub_labels", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/sub_labels")
 def get_sub_labels(split_joined: Optional[int] = None):
     try:
         events = Event.select(Event.sub_label).distinct()
@@ -642,7 +634,7 @@ def get_sub_labels(split_joined: Optional[int] = None):
     return JSONResponse(content=sub_labels)
 
 
-@router.get("/plus/models", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/plus/models")
 def plusModels(request: Request, filterByCurrentModelDetector: bool = False):
     if not request.app.frigate_config.plus_api.is_active():
         return JSONResponse(
@@ -684,9 +676,7 @@ def plusModels(request: Request, filterByCurrentModelDetector: bool = False):
     return JSONResponse(content=validModels)
 
 
-@router.get(
-    "/recognized_license_plates", dependencies=[Depends(allow_any_authenticated())]
-)
+@router.get("/recognized_license_plates")
 def get_recognized_license_plates(split_joined: Optional[int] = None):
     try:
         query = (
@@ -720,7 +710,7 @@ def get_recognized_license_plates(split_joined: Optional[int] = None):
     return JSONResponse(content=recognized_license_plates)
 
 
-@router.get("/timeline", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/timeline")
 def timeline(camera: str = "all", limit: int = 100, source_id: Optional[str] = None):
     clauses = []
 
@@ -757,7 +747,7 @@ def timeline(camera: str = "all", limit: int = 100, source_id: Optional[str] = N
     return JSONResponse(content=[t for t in timeline])
 
 
-@router.get("/timeline/hourly", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/timeline/hourly")
 def hourly_timeline(params: AppTimelineHourlyQueryParameters = Depends()):
     """Get hourly summary for timeline."""
     cameras = params.cameras

frigate/api/auth.py

@@ -32,178 +32,10 @@ from frigate.models import User
 
 logger = logging.getLogger(__name__)
 
-
-def require_admin_by_default():
-    """
-    Global admin requirement dependency for all endpoints by default.
-
-    This is set as the default dependency on the FastAPI app to ensure all
-    endpoints require admin access unless explicitly overridden with
-    allow_public(), allow_any_authenticated(), or require_role().
-
-    Port 5000 (internal) always has admin role set by the /auth endpoint,
-    so this check passes automatically for internal requests.
-
-    Certain paths are exempted from the global admin check because they must
-    be accessible before authentication (login, auth) or they have their own
-    route-level authorization dependencies that handle access control.
-    """
-    # Paths that have route-level auth dependencies and should bypass global admin check
-    # These paths still have authorization - it's handled by their route-level dependencies
-    EXEMPT_PATHS = {
-        # Public auth endpoints (allow_public)
-        "/auth",
-        "/auth/first_time_login",
-        "/login",
-        "/logout",
-        # Authenticated user endpoints (allow_any_authenticated)
-        "/profile",
-        # Public info endpoints (allow_public)
-        "/",
-        "/version",
-        "/config/schema.json",
-        # Authenticated user endpoints (allow_any_authenticated)
-        "/metrics",
-        "/stats",
-        "/stats/history",
-        "/config",
-        "/config/raw",
-        "/vainfo",
-        "/nvinfo",
-        "/labels",
-        "/sub_labels",
-        "/plus/models",
-        "/recognized_license_plates",
-        "/timeline",
-        "/timeline/hourly",
-        "/recordings/storage",
-        "/recordings/summary",
-        "/recordings/unavailable",
-        "/go2rtc/streams",
-        "/event_ids",
-        "/events",
-        "/exports",
-    }
-
-    # Path prefixes that should be exempt (for paths with parameters)
-    EXEMPT_PREFIXES = (
-        "/logs/",  # /logs/{service}
-        "/review",  # /review, /review/{id}, /review/summary, /review_ids, etc.
-        "/reviews/",  # /reviews/viewed, /reviews/delete
-        "/events/",  # /events/{id}/thumbnail, /events/summary, etc. (camera-scoped)
-        "/export/",  # /export/{camera}/start/..., /export/{id}/rename, /export/{id}
-        "/go2rtc/streams/",  # /go2rtc/streams/{camera}
-        "/users/",  # /users/{username}/password (has own auth)
-        "/preview/",  # /preview/{file}/thumbnail.jpg
-        "/exports/",  # /exports/{export_id}
-        "/vod/",  # /vod/{camera_name}/...
-        "/notifications/",  # /notifications/pubkey, /notifications/register
-    )
-
-    async def admin_checker(request: Request):
-        path = request.url.path
-
-        # Check exact path matches
-        if path in EXEMPT_PATHS:
-            return
-
-        # Check prefix matches for parameterized paths
-        if path.startswith(EXEMPT_PREFIXES):
-            return
-
-        # Dynamic camera path exemption:
-        # Any path whose first segment matches a configured camera name should
-        # bypass the global admin requirement. These endpoints enforce access
-        # via route-level dependencies (e.g. require_camera_access) to ensure
-        # per-camera authorization. This allows non-admin authenticated users
-        # (e.g. viewer role) to access camera-specific resources without
-        # needing admin privileges.
-        try:
-            if path.startswith("/"):
-                first_segment = path.split("/", 2)[1]
-                if (
-                    first_segment
-                    and first_segment in request.app.frigate_config.cameras
-                ):
-                    return
-        except Exception:
-            pass
-
-        # For all other paths, require admin role
-        # Port 5000 (internal) requests have admin role set automatically
-        role = request.headers.get("remote-role")
-        if role == "admin":
-            return
-
-        raise HTTPException(
-            status_code=403,
-            detail="Access denied. A user with the admin role is required.",
-        )
-
-    return admin_checker
-
-
-def _is_authenticated(request: Request) -> bool:
-    """
-    Helper to determine if a request is from an authenticated user.
-
-    Returns True if the request has a valid authenticated user (not anonymous).
-    Port 5000 internal requests are considered anonymous despite having admin role.
-    """
-    username = request.headers.get("remote-user")
-    return username is not None and username != "anonymous"
-
-
-def allow_public():
-    """
-    Override dependency to allow unauthenticated access to an endpoint.
-
-    Use this for endpoints that should be publicly accessible without
-    authentication, such as login page, health checks, or pre-auth info.
-
-    Example:
-        @router.get("/public-endpoint", dependencies=[Depends(allow_public())])
-    """
-
-    async def public_checker(request: Request):
-        return  # Always allow
-
-    return public_checker
-
-
-def allow_any_authenticated():
-    """
-    Override dependency to allow any authenticated user (bypass admin requirement).
-
-    Allows:
-    - Port 5000 internal requests (have admin role despite anonymous user)
-    - Any authenticated user with a real username (not "anonymous")
-
-    Rejects:
-    - Port 8971 requests with anonymous user (auth disabled, no proxy auth)
-
-    Example:
-        @router.get("/authenticated-endpoint", dependencies=[Depends(allow_any_authenticated())])
-    """
-
-    async def auth_checker(request: Request):
-        # Port 5000 requests have admin role and should be allowed
-        role = request.headers.get("remote-role")
-        if role == "admin":
-            return
-
-        # Otherwise require a real authenticated user (not anonymous)
-        if not _is_authenticated(request):
-            raise HTTPException(status_code=401, detail="Authentication required")
-        return
-
-    return auth_checker
-
-
 router = APIRouter(tags=[Tags.auth])
 
 
-@router.get("/auth/first_time_login", dependencies=[Depends(allow_public())])
+@router.get("/auth/first_time_login")
 def first_time_login(request: Request):
     """Return whether the admin first-time login help flag is set in config.
 
@@ -311,10 +143,7 @@ def get_jwt_secret() -> str:
             )
             jwt_secret = secrets.token_hex(64)
             try:
-                fd = os.open(
-                    jwt_secret_file, os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0o600
-                )
-                with os.fdopen(fd, "w") as f:
+                with open(jwt_secret_file, "w") as f:
                     f.write(str(jwt_secret))
             except Exception:
                 logger.warning(
@@ -359,35 +188,9 @@ def verify_password(password, password_hash):
     return secrets.compare_digest(password_hash, compare_hash)
 
 
-def validate_password_strength(password: str) -> tuple[bool, Optional[str]]:
-    """
-    Validate password strength.
-
-    Returns a tuple of (is_valid, error_message).
-    """
-    if not password:
-        return False, "Password cannot be empty"
-
-    if len(password) < 8:
-        return False, "Password must be at least 8 characters long"
-
-    if not any(c.isupper() for c in password):
-        return False, "Password must contain at least one uppercase letter"
-
-    if not any(c.isdigit() for c in password):
-        return False, "Password must contain at least one digit"
-
-    if not any(c in '!@#$%^&*(),.?":{}|<>' for c in password):
-        return False, "Password must contain at least one special character"
-
-    return True, None
-
-
 def create_encoded_jwt(user, role, expiration, secret):
     return jwt.encode(
-        {"alg": "HS256"},
-        {"sub": user, "role": role, "exp": expiration, "iat": int(time.time())},
-        secret,
+        {"alg": "HS256"}, {"sub": user, "role": role, "exp": expiration}, secret
     )
 
 
@@ -549,7 +352,7 @@ def resolve_role(
 
 
 # Endpoints
-@router.get("/auth", dependencies=[Depends(allow_public())])
+@router.get("/auth")
 def auth(request: Request):
     auth_config: AuthConfig = request.app.frigate_config.auth
     proxy_config: ProxyConfig = request.app.frigate_config.proxy
@@ -648,27 +451,13 @@ def auth(request: Request):
             return fail_response
 
         # if the jwt cookie is expiring soon
-        if jwt_source == "cookie" and expiration - JWT_REFRESH <= current_time:
+        elif jwt_source == "cookie" and expiration - JWT_REFRESH <= current_time:
             logger.debug("jwt token expiring soon, refreshing cookie")
-
-            # Check if password has been changed since token was issued
-            # If so, force re-login by rejecting the refresh
+            # ensure the user hasn't been deleted
             try:
-                user_obj = User.get_by_id(user)
-                if user_obj.password_changed_at is not None:
-                    token_iat = int(token.claims.get("iat", 0))
-                    password_changed_timestamp = int(
-                        user_obj.password_changed_at.timestamp()
-                    )
-                    if token_iat < password_changed_timestamp:
-                        logger.debug(
-                            "jwt token issued before password change, rejecting refresh"
-                        )
-                        return fail_response
+                User.get_by_id(user)
             except DoesNotExist:
-                logger.debug("user not found")
                 return fail_response
-
             new_expiration = current_time + JWT_SESSION_LENGTH
             new_encoded_jwt = create_encoded_jwt(
                 user, role, new_expiration, request.app.jwt_token
@@ -689,7 +478,7 @@ def auth(request: Request):
         return fail_response
 
 
-@router.get("/profile", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/profile")
 def profile(request: Request):
     username = request.headers.get("remote-user", "anonymous")
     role = request.headers.get("remote-role", "viewer")
@@ -703,7 +492,7 @@ def profile(request: Request):
     )
 
 
-@router.get("/logout", dependencies=[Depends(allow_public())])
+@router.get("/logout")
 def logout(request: Request):
     auth_config: AuthConfig = request.app.frigate_config.auth
     response = RedirectResponse("/login", status_code=303)
@@ -714,7 +503,7 @@ def logout(request: Request):
 limiter = Limiter(key_func=get_remote_addr)
 
 
-@router.post("/login", dependencies=[Depends(allow_public())])
+@router.post("/login")
 @limiter.limit(limit_value=rateLimiter.get_limit)
 def login(request: Request, body: AppPostLoginBody):
     JWT_COOKIE_NAME = request.app.frigate_config.auth.cookie_name
@@ -789,21 +578,13 @@ def create_user(
     return JSONResponse(content={"username": body.username})
 
 
-@router.delete("/users/{username}", dependencies=[Depends(require_role(["admin"]))])
-def delete_user(request: Request, username: str):
-    # Prevent deletion of the built-in admin user
-    if username == "admin":
-        return JSONResponse(
-            content={"message": "Cannot delete admin user"}, status_code=403
-        )
-
+@router.delete("/users/{username}")
+def delete_user(username: str):
     User.delete_by_id(username)
     return JSONResponse(content={"success": True})
 
 
-@router.put(
-    "/users/{username}/password", dependencies=[Depends(allow_any_authenticated())]
-)
+@router.put("/users/{username}/password")
 async def update_password(
     request: Request,
     username: str,
@@ -825,63 +606,10 @@ async def update_password(
 
     HASH_ITERATIONS = request.app.frigate_config.auth.hash_iterations
 
-    try:
-        user = User.get_by_id(username)
-    except DoesNotExist:
-        return JSONResponse(content={"message": "User not found"}, status_code=404)
-
-    # Require old_password when:
-    # 1. Non-admin user is changing another user's password (admin only action)
-    # 2. Any user is changing their own password
-    is_changing_own_password = current_username == username
-    is_non_admin = current_role != "admin"
-
-    if is_changing_own_password or is_non_admin:
-        if not body.old_password:
-            return JSONResponse(
-                content={"message": "Current password is required"},
-                status_code=400,
-            )
-        if not verify_password(body.old_password, user.password_hash):
-            return JSONResponse(
-                content={"message": "Current password is incorrect"},
-                status_code=401,
-            )
-
-    # Validate new password strength
-    is_valid, error_message = validate_password_strength(body.password)
-    if not is_valid:
-        return JSONResponse(
-            content={"message": error_message},
-            status_code=400,
-        )
-
     password_hash = hash_password(body.password, iterations=HASH_ITERATIONS)
-    User.update(
-        {
-            User.password_hash: password_hash,
-            User.password_changed_at: datetime.now(),
-        }
-    ).where(User.username == username).execute()
+    User.set_by_id(username, {User.password_hash: password_hash})
 
-    response = JSONResponse(content={"success": True})
-
-    # If user changed their own password, issue a new JWT to keep them logged in
-    if current_username == username:
-        JWT_COOKIE_NAME = request.app.frigate_config.auth.cookie_name
-        JWT_COOKIE_SECURE = request.app.frigate_config.auth.cookie_secure
-        JWT_SESSION_LENGTH = request.app.frigate_config.auth.session_length
-
-        expiration = int(time.time()) + JWT_SESSION_LENGTH
-        encoded_jwt = create_encoded_jwt(
-            username, current_role, expiration, request.app.jwt_token
-        )
-        # Set new JWT cookie on response
-        set_jwt_cookie(
-            response, JWT_COOKIE_NAME, encoded_jwt, expiration, JWT_COOKIE_SECURE
-        )
-
-    return response
+    return JSONResponse(content={"success": True})
 
 
 @router.put(

frigate/api/camera.py

@@ -15,11 +15,7 @@ from onvif import ONVIFCamera, ONVIFError
 from zeep.exceptions import Fault, TransportError
 from zeep.transports import AsyncTransport
 
-from frigate.api.auth import (
-    allow_any_authenticated,
-    require_camera_access,
-    require_role,
-)
+from frigate.api.auth import require_role
 from frigate.api.defs.tags import Tags
 from frigate.config.config import FrigateConfig
 from frigate.util.builtin import clean_camera_user_pass
@@ -54,7 +50,7 @@ def _is_valid_host(host: str) -> bool:
         return False
 
 
-@router.get("/go2rtc/streams", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/go2rtc/streams")
 def go2rtc_streams():
     r = requests.get("http://127.0.0.1:1984/api/streams")
     if not r.ok:
@@ -70,9 +66,7 @@ def go2rtc_streams():
     return JSONResponse(content=stream_data)
 
 
-@router.get(
-    "/go2rtc/streams/{camera_name}", dependencies=[Depends(require_camera_access)]
-)
+@router.get("/go2rtc/streams/{camera_name}")
 def go2rtc_camera_stream(request: Request, camera_name: str):
     r = requests.get(
         f"http://127.0.0.1:1984/api/streams?src={camera_name}&video=all&audio=all&microphone"
@@ -167,7 +161,7 @@ def go2rtc_delete_stream(stream_name: str):
         )
 
 
-@router.get("/ffprobe", dependencies=[Depends(require_role(["admin"]))])
+@router.get("/ffprobe")
 def ffprobe(request: Request, paths: str = "", detailed: bool = False):
     path_param = paths
 

frigate/api/classification.py

@@ -710,7 +710,7 @@ def delete_classification_dataset_images(
         if os.path.isfile(file_path):
             os.unlink(file_path)
 
-    if os.path.exists(folder) and not os.listdir(folder) and category.lower() != "none":
+    if os.path.exists(folder) and not os.listdir(folder):
         os.rmdir(folder)
 
     return JSONResponse(
@@ -870,46 +870,6 @@ def categorize_classification_image(request: Request, name: str, body: dict = No
     )
 
 
-@router.post(
-    "/classification/{name}/dataset/{category}/create",
-    response_model=GenericResponse,
-    dependencies=[Depends(require_role(["admin"]))],
-    summary="Create an empty classification category folder",
-    description="""Creates an empty folder for a classification category.
-    This is used to create folders for categories that don't have images yet.
-    Returns a success message or an error if the name is invalid.""",
-)
-def create_classification_category(request: Request, name: str, category: str):
-    config: FrigateConfig = request.app.frigate_config
-
-    if name not in config.classification.custom:
-        return JSONResponse(
-            content=(
-                {
-                    "success": False,
-                    "message": f"{name} is not a known classification model.",
-                }
-            ),
-            status_code=404,
-        )
-
-    category_folder = os.path.join(
-        CLIPS_DIR, sanitize_filename(name), "dataset", sanitize_filename(category)
-    )
-
-    os.makedirs(category_folder, exist_ok=True)
-
-    return JSONResponse(
-        content=(
-            {
-                "success": True,
-                "message": f"Successfully created category folder: {category}",
-            }
-        ),
-        status_code=200,
-    )
-
-
 @router.post(
     "/classification/{name}/train/delete",
     response_model=GenericResponse,

frigate/api/defs/request/app_body.py

@@ -11,7 +11,6 @@ class AppConfigSetBody(BaseModel):
 
 class AppPutPasswordBody(BaseModel):
     password: str
-    old_password: Optional[str] = None
 
 
 class AppPostUsersBody(BaseModel):

frigate/api/defs/request/events_body.py

@@ -29,6 +29,7 @@ class EventsDescriptionBody(BaseModel):
 
 
 class EventsCreateBody(BaseModel):
+    source_type: Optional[str] = "api"
     sub_label: Optional[str] = None
     score: Optional[float] = 0
     duration: Optional[int] = 30

frigate/api/event.py

@@ -22,7 +22,6 @@ from peewee import JOIN, DoesNotExist, fn, operator
 from playhouse.shortcuts import model_to_dict
 
 from frigate.api.auth import (
-    allow_any_authenticated,
     get_allowed_cameras_for_filter,
     require_camera_access,
     require_role,
@@ -70,7 +69,6 @@ router = APIRouter(tags=[Tags.events])
 @router.get(
     "/events",
     response_model=list[EventResponse],
-    dependencies=[Depends(allow_any_authenticated())],
     summary="Get events",
     description="Returns a list of events.",
 )
@@ -345,8 +343,7 @@ def events(
 @router.get(
     "/events/explore",
     response_model=list[EventResponse],
-    dependencies=[Depends(allow_any_authenticated())],
-    summary="Get summary of objects",
+    summary="Get summary of objects.",
     description="""Gets a summary of objects from the database.
     Returns a list of objects with a max of `limit` objects for each label.
     """,
@@ -438,8 +435,7 @@ def events_explore(
 @router.get(
     "/event_ids",
     response_model=list[EventResponse],
-    dependencies=[Depends(allow_any_authenticated())],
-    summary="Get events by ids",
+    summary="Get events by ids.",
     description="""Gets events by a list of ids.
     Returns a list of events.
     """,
@@ -472,8 +468,7 @@ async def event_ids(ids: str, request: Request):
 
 @router.get(
     "/events/search",
-    dependencies=[Depends(allow_any_authenticated())],
-    summary="Search events",
+    summary="Search events.",
     description="""Searches for events in the database.
     Returns a list of events.
     """,
@@ -813,7 +808,7 @@ def events_search(
     return JSONResponse(content=processed_events)
 
 
-@router.get("/events/summary", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/events/summary")
 def events_summary(
     params: EventsSummaryQueryParams = Depends(),
     allowed_cameras: List[str] = Depends(get_allowed_cameras_for_filter),
@@ -923,8 +918,7 @@ def events_summary(
 @router.get(
     "/events/{event_id}",
     response_model=EventResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-    summary="Get event by id",
+    summary="Get event by id.",
     description="Gets an event by its id.",
 )
 async def event(event_id: str, request: Request):
@@ -967,8 +961,7 @@ def set_retain(event_id: str):
 @router.post(
     "/events/{event_id}/plus",
     response_model=EventUploadPlusResponse,
-    dependencies=[Depends(require_role(["admin"]))],
-    summary="Send event to Frigate+",
+    summary="Send event to Frigate+.",
     description="""Sends an event to Frigate+.
     Returns a success message or an error if the event is not found.
     """,
@@ -1108,7 +1101,6 @@ async def send_to_plus(request: Request, event_id: str, body: SubmitPlusBody = N
 @router.put(
     "/events/{event_id}/false_positive",
     response_model=EventUploadPlusResponse,
-    dependencies=[Depends(require_role(["admin"]))],
     summary="Submit false positive to Frigate+",
     description="""Submit an event as a false positive to Frigate+.
     This endpoint is the same as the standard Frigate+ submission endpoint,
@@ -1207,7 +1199,7 @@ async def false_positive(request: Request, event_id: str):
     "/events/{event_id}/retain",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Stop event from being retained indefinitely",
+    summary="Stop event from being retained indefinitely.",
     description="""Stops an event from being retained indefinitely.
     Returns a success message or an error if the event is not found.
     NOTE: This is a legacy endpoint and is not supported in the frontend.
@@ -1236,7 +1228,7 @@ async def delete_retain(event_id: str, request: Request):
     "/events/{event_id}/sub_label",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Set event sub label",
+    summary="Set event sub label.",
     description="""Sets an event's sub label.
     Returns a success message or an error if the event is not found.
     """,
@@ -1295,7 +1287,7 @@ async def set_sub_label(
     "/events/{event_id}/recognized_license_plate",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Set event license plate",
+    summary="Set event license plate.",
     description="""Sets an event's license plate.
     Returns a success message or an error if the event is not found.
     """,
@@ -1355,7 +1347,7 @@ async def set_plate(
     "/events/{event_id}/description",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Set event description",
+    summary="Set event description.",
     description="""Sets an event's description.
     Returns a success message or an error if the event is not found.
     """,
@@ -1411,7 +1403,7 @@ async def set_description(
     "/events/{event_id}/description/regenerate",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Regenerate event description",
+    summary="Regenerate event description.",
     description="""Regenerates an event's description.
     Returns a success message or an error if the event is not found.
     """,
@@ -1463,8 +1455,8 @@ async def regenerate_description(
 @router.post(
     "/description/generate",
     response_model=GenericResponse,
-    dependencies=[Depends(require_role(["admin"]))],
-    summary="Generate description embedding",
+    # dependencies=[Depends(require_role(["admin"]))],
+    summary="Generate description embedding.",
     description="""Generates an embedding for an event's description.
     Returns a success message or an error if the event is not found.
     """,
@@ -1529,7 +1521,7 @@ async def delete_single_event(event_id: str, request: Request) -> dict:
     "/events/{event_id}",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Delete event",
+    summary="Delete event.",
     description="""Deletes an event from the database.
     Returns a success message or an error if the event is not found.
     """,
@@ -1544,7 +1536,7 @@ async def delete_event(request: Request, event_id: str):
     "/events/",
     response_model=EventMultiDeleteResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Delete events",
+    summary="Delete events.",
     description="""Deletes a list of events from the database.
     Returns a success message or an error if the events are not found.
     """,
@@ -1578,7 +1570,7 @@ async def delete_events(request: Request, body: EventsDeleteBody):
     "/events/{camera_name}/{label}/create",
     response_model=EventCreateResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Create manual event",
+    summary="Create manual event.",
     description="""Creates a manual event in the database.
     Returns a success message or an error if the event is not found.
     NOTES:
@@ -1620,7 +1612,7 @@ def create_event(
             body.score,
             body.sub_label,
             body.duration,
-            "api",
+            body.source_type,
             body.draw,
         ),
         EventMetadataTypeEnum.manual_event_create.value,
@@ -1642,7 +1634,7 @@ def create_event(
     "/events/{event_id}/end",
     response_model=GenericResponse,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="End manual event",
+    summary="End manual event.",
     description="""Ends a manual event.
     Returns a success message or an error if the event is not found.
     NOTE: This should only be used for manual events.
@@ -1652,27 +1644,10 @@ async def end_event(request: Request, event_id: str, body: EventsEndBody):
     try:
         event: Event = Event.get(Event.id == event_id)
         await require_camera_access(event.camera, request=request)
-
-        if body.end_time is not None and body.end_time < event.start_time:
-            return JSONResponse(
-                content=(
-                    {
-                        "success": False,
-                        "message": f"end_time ({body.end_time}) cannot be before start_time ({event.start_time}).",
-                    }
-                ),
-                status_code=400,
-            )
-
         end_time = body.end_time or datetime.datetime.now().timestamp()
         request.app.event_metadata_updater.publish(
             (event_id, end_time), EventMetadataTypeEnum.manual_event_end.value
         )
-    except DoesNotExist:
-        return JSONResponse(
-            content=({"success": False, "message": f"Event {event_id} not found."}),
-            status_code=404,
-        )
     except Exception:
         return JSONResponse(
             content=(
@@ -1691,7 +1666,7 @@ async def end_event(request: Request, event_id: str, body: EventsEndBody):
     "/trigger/embedding",
     response_model=dict,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Create trigger embedding",
+    summary="Create trigger embedding.",
     description="""Creates a trigger embedding for a specific trigger.
     Returns a success message or an error if the trigger is not found.
     """,
@@ -1748,40 +1723,37 @@ def create_trigger_embedding(
             if event.data.get("type") != "object":
                 return
 
-            # Get the thumbnail
-            thumbnail = get_event_thumbnail_bytes(event)
-
-            if thumbnail is None:
-                return JSONResponse(
-                    content={
-                        "success": False,
-                        "message": f"Failed to get thumbnail for {body.data} for {body.type} trigger",
-                    },
-                    status_code=400,
+            if thumbnail := get_event_thumbnail_bytes(event):
+                cursor = context.db.execute_sql(
+                    """
+                    SELECT thumbnail_embedding FROM vec_thumbnails WHERE id = ?
+                    """,
+                    [body.data],
                 )
 
-            # Try to reuse existing embedding from database
-            cursor = context.db.execute_sql(
-                """
-                SELECT thumbnail_embedding FROM vec_thumbnails WHERE id = ?
-                """,
-                [body.data],
-            )
+                row = cursor.fetchone() if cursor else None
 
-            row = cursor.fetchone() if cursor else None
-
-            if row:
-                query_embedding = row[0]
-                embedding = np.frombuffer(query_embedding, dtype=np.float32)
+                if row:
+                    query_embedding = row[0]
+                    embedding = np.frombuffer(query_embedding, dtype=np.float32)
             else:
-                # Generate new embedding
+                # Extract valid thumbnail
+                thumbnail = get_event_thumbnail_bytes(event)
+
+                if thumbnail is None:
+                    return JSONResponse(
+                        content={
+                            "success": False,
+                            "message": f"Failed to get thumbnail for {body.data} for {body.type} trigger",
+                        },
+                        status_code=400,
+                    )
+
                 embedding = context.generate_image_embedding(
                     body.data, (base64.b64encode(thumbnail).decode("ASCII"))
                 )
 
-        if embedding is None or (
-            isinstance(embedding, (list, np.ndarray)) and len(embedding) == 0
-        ):
+        if not embedding:
             return JSONResponse(
                 content={
                     "success": False,
@@ -1849,7 +1821,7 @@ def create_trigger_embedding(
     "/trigger/embedding/{camera_name}/{name}",
     response_model=dict,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Update trigger embedding",
+    summary="Update trigger embedding.",
     description="""Updates a trigger embedding for a specific trigger.
     Returns a success message or an error if the trigger is not found.
     """,
@@ -1916,9 +1888,7 @@ def update_trigger_embedding(
                 body.data, (base64.b64encode(thumbnail).decode("ASCII"))
             )
 
-        if embedding is None or (
-            isinstance(embedding, (list, np.ndarray)) and len(embedding) == 0
-        ):
+        if not embedding:
             return JSONResponse(
                 content={
                     "success": False,
@@ -2014,7 +1984,7 @@ def update_trigger_embedding(
     "/trigger/embedding/{camera_name}/{name}",
     response_model=dict,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Delete trigger embedding",
+    summary="Delete trigger embedding.",
     description="""Deletes a trigger embedding for a specific trigger.
     Returns a success message or an error if the trigger is not found.
     """,
@@ -2088,7 +2058,7 @@ def delete_trigger_embedding(
     "/triggers/status/{camera_name}",
     response_model=dict,
     dependencies=[Depends(require_role(["admin"]))],
-    summary="Get triggers status",
+    summary="Get triggers status.",
     description="""Gets the status of all triggers for a specific camera.
     Returns a success message or an error if the camera is not found.
     """,

frigate/api/export.py

@@ -14,7 +14,6 @@ from peewee import DoesNotExist
 from playhouse.shortcuts import model_to_dict
 
 from frigate.api.auth import (
-    allow_any_authenticated,
     get_allowed_cameras_for_filter,
     require_camera_access,
     require_role,
@@ -45,7 +44,6 @@ router = APIRouter(tags=[Tags.export])
 @router.get(
     "/exports",
     response_model=ExportsResponse,
-    dependencies=[Depends(allow_any_authenticated())],
     summary="Get exports",
     description="""Gets all exports from the database for cameras the user has access to.
     Returns a list of exports ordered by date (most recent first).""",
@@ -274,7 +272,6 @@ async def export_delete(event_id: str, request: Request):
 @router.get(
     "/exports/{export_id}",
     response_model=ExportModel,
-    dependencies=[Depends(allow_any_authenticated())],
     summary="Get a single export",
     description="""Gets a specific export by ID. The user must have access to the camera
     associated with the export.""",

frigate/api/fastapi_app.py

@@ -2,7 +2,7 @@ import logging
 import re
 from typing import Optional
 
-from fastapi import Depends, FastAPI, Request
+from fastapi import FastAPI, Request
 from fastapi.responses import JSONResponse
 from joserfc.jwk import OctKey
 from playhouse.sqliteq import SqliteQueueDatabase
@@ -24,7 +24,7 @@ from frigate.api import (
     preview,
     review,
 )
-from frigate.api.auth import get_jwt_secret, limiter, require_admin_by_default
+from frigate.api.auth import get_jwt_secret, limiter
 from frigate.comms.event_metadata_updater import (
     EventMetadataPublisher,
 )
@@ -62,15 +62,11 @@ def create_fastapi_app(
     stats_emitter: StatsEmitter,
     event_metadata_updater: EventMetadataPublisher,
     config_publisher: CameraConfigUpdatePublisher,
-    enforce_default_admin: bool = True,
 ):
     logger.info("Starting FastAPI app")
     app = FastAPI(
         debug=False,
         swagger_ui_parameters={"apisSorter": "alpha", "operationsSorter": "alpha"},
-        dependencies=[Depends(require_admin_by_default())]
-        if enforce_default_admin
-        else [],
     )
 
     # update the request_address with the x-forwarded-for header from nginx

frigate/api/media.py

@@ -22,11 +22,7 @@ from pathvalidate import sanitize_filename
 from peewee import DoesNotExist, fn, operator
 from tzlocal import get_localzone_name
 
-from frigate.api.auth import (
-    allow_any_authenticated,
-    get_allowed_cameras_for_filter,
-    require_camera_access,
-)
+from frigate.api.auth import get_allowed_cameras_for_filter, require_camera_access
 from frigate.api.defs.query.media_query_parameters import (
     Extension,
     MediaEventsSnapshotQueryParams,
@@ -397,7 +393,7 @@ async def submit_recording_snapshot_to_plus(
         )
 
 
-@router.get("/recordings/storage", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/recordings/storage")
 def get_recordings_storage_usage(request: Request):
     recording_stats = request.app.stats_emitter.get_latest_stats()["service"][
         "storage"
@@ -421,7 +417,7 @@ def get_recordings_storage_usage(request: Request):
     return JSONResponse(content=camera_usages)
 
 
-@router.get("/recordings/summary", dependencies=[Depends(allow_any_authenticated())])
+@router.get("/recordings/summary")
 def all_recordings_summary(
     request: Request,
     params: MediaRecordingsSummaryQueryParams = Depends(),
@@ -639,11 +635,7 @@ async def recordings(
     return JSONResponse(content=list(recordings))
 
 
-@router.get(
-    "/recordings/unavailable",
-    response_model=list[dict],
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/recordings/unavailable", response_model=list[dict])
 async def no_recordings(
     request: Request,
     params: MediaRecordingsAvailabilityQueryParams = Depends(),
@@ -837,19 +829,7 @@ async def recording_clip(
     dependencies=[Depends(require_camera_access)],
     description="Returns an HLS playlist for the specified timestamp-range on the specified camera. Append /master.m3u8 or /index.m3u8 for HLS playback.",
 )
-async def vod_ts(
-    camera_name: str,
-    start_ts: float,
-    end_ts: float,
-    force_discontinuity: bool = False,
-):
-    logger.debug(
-        "VOD: Generating VOD for %s from %s to %s with force_discontinuity=%s",
-        camera_name,
-        start_ts,
-        end_ts,
-        force_discontinuity,
-    )
+async def vod_ts(camera_name: str, start_ts: float, end_ts: float):
     recordings = (
         Recordings.select(
             Recordings.path,
@@ -874,14 +854,6 @@ async def vod_ts(
 
     recording: Recordings
     for recording in recordings:
-        logger.debug(
-            "VOD: processing recording: %s start=%s end=%s duration=%s",
-            recording.path,
-            recording.start_time,
-            recording.end_time,
-            recording.duration,
-        )
-
         clip = {"type": "source", "path": recording.path}
         duration = int(recording.duration * 1000)
 
@@ -890,11 +862,6 @@ async def vod_ts(
             inpoint = int((start_ts - recording.start_time) * 1000)
             clip["clipFrom"] = inpoint
             duration -= inpoint
-            logger.debug(
-                "VOD: applied clipFrom %sms to %s",
-                inpoint,
-                recording.path,
-            )
 
         # adjust end if recording.end_time is after end_ts
         if recording.end_time > end_ts:
@@ -902,23 +869,12 @@ async def vod_ts(
 
         if duration < min_duration_ms:
             # skip if the clip has no valid duration (too short to contain frames)
-            logger.debug(
-                "VOD: skipping recording %s - resulting duration %sms too short",
-                recording.path,
-                duration,
-            )
             continue
 
         if min_duration_ms <= duration < max_duration_ms:
             clip["keyFrameDurations"] = [duration]
             clips.append(clip)
             durations.append(duration)
-            logger.debug(
-                "VOD: added clip %s duration_ms=%s clipFrom=%s",
-                recording.path,
-                duration,
-                clip.get("clipFrom"),
-            )
         else:
             logger.warning(f"Recording clip is missing or empty: {recording.path}")
 
@@ -938,7 +894,7 @@ async def vod_ts(
     return JSONResponse(
         content={
             "cache": hour_ago.timestamp() > start_ts,
-            "discontinuity": force_discontinuity,
+            "discontinuity": False,
             "consistentSequenceMediaInfo": True,
             "durations": durations,
             "segment_duration": max(durations),
@@ -981,7 +937,6 @@ async def vod_hour(
 
 @router.get(
     "/vod/event/{event_id}",
-    dependencies=[Depends(allow_any_authenticated())],
     description="Returns an HLS playlist for the specified object. Append /master.m3u8 or /index.m3u8 for HLS playback.",
 )
 async def vod_event(
@@ -1022,19 +977,6 @@ async def vod_event(
     return vod_response
 
 
-@router.get(
-    "/vod/clip/{camera_name}/start/{start_ts}/end/{end_ts}",
-    dependencies=[Depends(require_camera_access)],
-    description="Returns an HLS playlist for a timestamp range with HLS discontinuity enabled. Append /master.m3u8 or /index.m3u8 for HLS playback.",
-)
-async def vod_clip(
-    camera_name: str,
-    start_ts: float,
-    end_ts: float,
-):
-    return await vod_ts(camera_name, start_ts, end_ts, force_discontinuity=True)
-
-
 @router.get(
     "/events/{event_id}/snapshot.jpg",
     description="Returns a snapshot image for the specified object id. NOTE: The query params only take affect while the event is in-progress. Once the event has ended the snapshot configuration is used.",
@@ -1111,10 +1053,7 @@ async def event_snapshot(
     )
 
 
-@router.get(
-    "/events/{event_id}/thumbnail.{extension}",
-    dependencies=[Depends(require_camera_access)],
-)
+@router.get("/events/{event_id}/thumbnail.{extension}")
 async def event_thumbnail(
     request: Request,
     event_id: str,
@@ -1312,10 +1251,7 @@ def grid_snapshot(
         )
 
 
-@router.get(
-    "/events/{event_id}/snapshot-clean.webp",
-    dependencies=[Depends(require_camera_access)],
-)
+@router.get("/events/{event_id}/snapshot-clean.webp")
 def event_snapshot_clean(request: Request, event_id: str, download: bool = False):
     webp_bytes = None
     try:
@@ -1439,9 +1375,7 @@ def event_snapshot_clean(request: Request, event_id: str, download: bool = False
     )
 
 
-@router.get(
-    "/events/{event_id}/clip.mp4", dependencies=[Depends(require_camera_access)]
-)
+@router.get("/events/{event_id}/clip.mp4")
 async def event_clip(
     request: Request,
     event_id: str,
@@ -1469,9 +1403,7 @@ async def event_clip(
     )
 
 
-@router.get(
-    "/events/{event_id}/preview.gif", dependencies=[Depends(require_camera_access)]
-)
+@router.get("/events/{event_id}/preview.gif")
 def event_preview(request: Request, event_id: str):
     try:
         event: Event = Event.get(Event.id == event_id)
@@ -1824,7 +1756,7 @@ def preview_mp4(
     )
 
 
-@router.get("/review/{event_id}/preview", dependencies=[Depends(require_camera_access)])
+@router.get("/review/{event_id}/preview")
 def review_preview(
     request: Request,
     event_id: str,
@@ -1850,12 +1782,8 @@ def review_preview(
         return preview_mp4(request, review.camera, start_ts, end_ts)
 
 
-@router.get(
-    "/preview/{file_name}/thumbnail.jpg", dependencies=[Depends(require_camera_access)]
-)
-@router.get(
-    "/preview/{file_name}/thumbnail.webp", dependencies=[Depends(require_camera_access)]
-)
+@router.get("/preview/{file_name}/thumbnail.jpg")
+@router.get("/preview/{file_name}/thumbnail.webp")
 def preview_thumbnail(file_name: str):
     """Get a thumbnail from the cached preview frames."""
     if len(file_name) > 1000:

frigate/api/notification.py

@@ -5,12 +5,11 @@ import os
 from typing import Any
 
 from cryptography.hazmat.primitives import serialization
-from fastapi import APIRouter, Depends, Request
+from fastapi import APIRouter, Request
 from fastapi.responses import JSONResponse
 from peewee import DoesNotExist
 from py_vapid import Vapid01, utils
 
-from frigate.api.auth import allow_any_authenticated
 from frigate.api.defs.tags import Tags
 from frigate.const import CONFIG_DIR
 from frigate.models import User
@@ -22,7 +21,6 @@ router = APIRouter(tags=[Tags.notifications])
 
 @router.get(
     "/notifications/pubkey",
-    dependencies=[Depends(allow_any_authenticated())],
     summary="Get VAPID public key",
     description="""Gets the VAPID public key for the notifications.
     Returns the public key or an error if notifications are not enabled.
@@ -49,7 +47,6 @@ def get_vapid_pub_key(request: Request):
 
 @router.post(
     "/notifications/register",
-    dependencies=[Depends(allow_any_authenticated())],
     summary="Register notifications",
     description="""Registers a notifications subscription.
     Returns a success message or an error if the subscription is not provided.

frigate/api/preview.py

@@ -5,14 +5,10 @@ import os
 from datetime import datetime, timedelta, timezone
 
 import pytz
-from fastapi import APIRouter, Depends, HTTPException
+from fastapi import APIRouter, Depends
 from fastapi.responses import JSONResponse
 
-from frigate.api.auth import (
-    allow_any_authenticated,
-    get_allowed_cameras_for_filter,
-    require_camera_access,
-)
+from frigate.api.auth import require_camera_access
 from frigate.api.defs.response.preview_response import (
     PreviewFramesResponse,
     PreviewsResponse,
@@ -30,32 +26,19 @@ router = APIRouter(tags=[Tags.preview])
 @router.get(
     "/preview/{camera_name}/start/{start_ts}/end/{end_ts}",
     response_model=PreviewsResponse,
-    dependencies=[Depends(allow_any_authenticated())],
+    dependencies=[Depends(require_camera_access)],
     summary="Get preview clips for time range",
     description="""Gets all preview clips for a specified camera and time range.
     Returns a list of preview video clips that overlap with the requested time period,
     ordered by start time. Use camera_name='all' to get previews from all cameras.
     Returns an error if no previews are found.""",
 )
-def preview_ts(
-    camera_name: str,
-    start_ts: float,
-    end_ts: float,
-    allowed_cameras: list[str] = Depends(get_allowed_cameras_for_filter),
-):
+def preview_ts(camera_name: str, start_ts: float, end_ts: float):
     """Get all mp4 previews relevant for time period."""
     if camera_name != "all":
-        if camera_name not in allowed_cameras:
-            raise HTTPException(status_code=403, detail="Access denied for camera")
-        camera_list = [camera_name]
+        camera_clause = Previews.camera == camera_name
     else:
-        camera_list = allowed_cameras
-
-    if not camera_list:
-        return JSONResponse(
-            content={"success": False, "message": "No previews found."},
-            status_code=404,
-        )
+        camera_clause = True
 
     previews = (
         Previews.select(
@@ -70,7 +53,7 @@ def preview_ts(
             | Previews.end_time.between(start_ts, end_ts)
             | ((start_ts > Previews.start_time) & (end_ts < Previews.end_time))
         )
-        .where(Previews.camera << camera_list)
+        .where(camera_clause)
         .order_by(Previews.start_time.asc())
         .dicts()
         .iterator()
@@ -105,21 +88,14 @@ def preview_ts(
 @router.get(
     "/preview/{year_month}/{day}/{hour}/{camera_name}/{tz_name}",
     response_model=PreviewsResponse,
-    dependencies=[Depends(allow_any_authenticated())],
+    dependencies=[Depends(require_camera_access)],
     summary="Get preview clips for specific hour",
     description="""Gets all preview clips for a specific hour in a given timezone.
     Converts the provided date/time from the specified timezone to UTC and retrieves
     all preview clips for that hour. Use camera_name='all' to get previews from all cameras.
     The tz_name should be a timezone like 'America/New_York' (use commas instead of slashes).""",
 )
-def preview_hour(
-    year_month: str,
-    day: int,
-    hour: int,
-    camera_name: str,
-    tz_name: str,
-    allowed_cameras: list[str] = Depends(get_allowed_cameras_for_filter),
-):
+def preview_hour(year_month: str, day: int, hour: int, camera_name: str, tz_name: str):
     """Get all mp4 previews relevant for time period given the timezone"""
     parts = year_month.split("-")
     start_date = (
@@ -130,7 +106,7 @@ def preview_hour(
     start_ts = start_date.timestamp()
     end_ts = end_date.timestamp()
 
-    return preview_ts(camera_name, start_ts, end_ts, allowed_cameras)
+    return preview_ts(camera_name, start_ts, end_ts)
 
 
 @router.get(

frigate/api/review.py

@@ -14,7 +14,6 @@ from peewee import Case, DoesNotExist, IntegrityError, fn, operator
 from playhouse.shortcuts import model_to_dict
 
 from frigate.api.auth import (
-    allow_any_authenticated,
     get_allowed_cameras_for_filter,
     get_current_user,
     require_camera_access,
@@ -44,11 +43,7 @@ logger = logging.getLogger(__name__)
 router = APIRouter(tags=[Tags.review])
 
 
-@router.get(
-    "/review",
-    response_model=list[ReviewSegmentResponse],
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/review", response_model=list[ReviewSegmentResponse])
 async def review(
     params: ReviewQueryParams = Depends(),
     current_user: dict = Depends(get_current_user),
@@ -157,11 +152,7 @@ async def review(
     return JSONResponse(content=[r for r in review_query])
 
 
-@router.get(
-    "/review_ids",
-    response_model=list[ReviewSegmentResponse],
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/review_ids", response_model=list[ReviewSegmentResponse])
 async def review_ids(request: Request, ids: str):
     ids = ids.split(",")
 
@@ -195,11 +186,7 @@ async def review_ids(request: Request, ids: str):
         )
 
 
-@router.get(
-    "/review/summary",
-    response_model=ReviewSummaryResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/review/summary", response_model=ReviewSummaryResponse)
 async def review_summary(
     params: ReviewSummaryQueryParams = Depends(),
     current_user: dict = Depends(get_current_user),
@@ -474,11 +461,7 @@ async def review_summary(
     return JSONResponse(content=data)
 
 
-@router.post(
-    "/reviews/viewed",
-    response_model=GenericResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.post("/reviews/viewed", response_model=GenericResponse)
 async def set_multiple_reviewed(
     request: Request,
     body: ReviewModifyMultipleBody,
@@ -577,9 +560,7 @@ def delete_reviews(body: ReviewModifyMultipleBody):
 
 
 @router.get(
-    "/review/activity/motion",
-    response_model=list[ReviewActivityMotionResponse],
-    dependencies=[Depends(allow_any_authenticated())],
+    "/review/activity/motion", response_model=list[ReviewActivityMotionResponse]
 )
 def motion_activity(
     params: ReviewActivityMotionQueryParams = Depends(),
@@ -663,11 +644,7 @@ def motion_activity(
     return JSONResponse(content=normalized)
 
 
-@router.get(
-    "/review/event/{event_id}",
-    response_model=ReviewSegmentResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/review/event/{event_id}", response_model=ReviewSegmentResponse)
 async def get_review_from_event(request: Request, event_id: str):
     try:
         review = ReviewSegment.get(
@@ -682,11 +659,7 @@ async def get_review_from_event(request: Request, event_id: str):
         )
 
 
-@router.get(
-    "/review/{review_id}",
-    response_model=ReviewSegmentResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.get("/review/{review_id}", response_model=ReviewSegmentResponse)
 async def get_review(request: Request, review_id: str):
     try:
         review = ReviewSegment.get(ReviewSegment.id == review_id)
@@ -699,11 +672,7 @@ async def get_review(request: Request, review_id: str):
         )
 
 
-@router.delete(
-    "/review/{review_id}/viewed",
-    response_model=GenericResponse,
-    dependencies=[Depends(allow_any_authenticated())],
-)
+@router.delete("/review/{review_id}/viewed", response_model=GenericResponse)
 async def set_not_reviewed(
     review_id: str,
     current_user: dict = Depends(get_current_user),
@@ -741,7 +710,6 @@ async def set_not_reviewed(
 
 @router.post(
     "/review/summarize/start/{start_ts}/end/{end_ts}",
-    dependencies=[Depends(allow_any_authenticated())],
     description="Use GenAI to summarize review items over a period of time.",
 )
 def generate_review_summary(request: Request, start_ts: float, end_ts: float):

frigate/comms/dispatcher.py

@@ -607,27 +607,23 @@ class Dispatcher:
         )
         self.publish(f"{camera_name}/snapshots/state", payload, retain=True)
 
-    def _on_ptz_command(self, camera_name: str, payload: str | bytes) -> None:
+    def _on_ptz_command(self, camera_name: str, payload: str) -> None:
         """Callback for ptz topic."""
         try:
-            preset: str = (
-                payload.decode("utf-8") if isinstance(payload, bytes) else payload
-            ).lower()
-
-            if "preset" in preset:
+            if "preset" in payload.lower():
                 command = OnvifCommandEnum.preset
-                param = preset[preset.index("_") + 1 :]
-            elif "move_relative" in preset:
+                param = payload.lower()[payload.index("_") + 1 :]
+            elif "move_relative" in payload.lower():
                 command = OnvifCommandEnum.move_relative
-                param = preset[preset.index("_") + 1 :]
+                param = payload.lower()[payload.index("_") + 1 :]
             else:
-                command = OnvifCommandEnum[preset]
+                command = OnvifCommandEnum[payload.lower()]
                 param = ""
 
             self.onvif.handle_command(camera_name, command, param)
             logger.info(f"Setting ptz command to {command} for {camera_name}")
         except KeyError as k:
-            logger.error(f"Invalid PTZ command {preset}: {k}")
+            logger.error(f"Invalid PTZ command {payload}: {k}")
 
     def _on_birdseye_command(self, camera_name: str, payload: str) -> None:
         """Callback for birdseye topic."""

frigate/comms/webpush.py

@@ -21,7 +21,7 @@ from frigate.config.camera.updater import (
     CameraConfigUpdateEnum,
     CameraConfigUpdateSubscriber,
 )
-from frigate.const import BASE_DIR, CONFIG_DIR
+from frigate.const import CONFIG_DIR
 from frigate.models import User
 
 logger = logging.getLogger(__name__)
@@ -371,39 +371,14 @@ class WebPushClient(Communicator):
 
         sorted_objects.update(payload["after"]["data"]["sub_labels"])
 
-        image = f"{payload['after']['thumb_path'].replace(BASE_DIR, '')}"
+        image = f"{payload['after']['thumb_path'].replace('/media/frigate', '')}"
         ended = state == "end" or state == "genai"
 
         if state == "genai" and payload["after"]["data"]["metadata"]:
-            base_title = payload["after"]["data"]["metadata"]["title"]
-            threat_level = payload["after"]["data"]["metadata"].get(
-                "potential_threat_level", 0
-            )
-
-            # Add prefix for threat levels 1 and 2
-            if threat_level == 1:
-                title = f"Needs Review: {base_title}"
-            elif threat_level == 2:
-                title = f"Security Concern: {base_title}"
-            else:
-                title = base_title
-
+            title = payload["after"]["data"]["metadata"]["title"]
             message = payload["after"]["data"]["metadata"]["scene"]
         else:
-            zone_names = payload["after"]["data"]["zones"]
-            formatted_zone_names = []
-
-            for zone_name in zone_names:
-                if zone_name in self.config.cameras[camera].zones:
-                    formatted_zone_names.append(
-                        self.config.cameras[camera]
-                        .zones[zone_name]
-                        .get_formatted_name(zone_name)
-                    )
-                else:
-                    formatted_zone_names.append(titlecase(zone_name.replace("_", " ")))
-
-            title = f"{titlecase(', '.join(sorted_objects).replace('_', ' '))}{' was' if state == 'end' else ''} detected in {', '.join(formatted_zone_names)}"
+            title = f"{titlecase(', '.join(sorted_objects).replace('_', ' '))}{' was' if state == 'end' else ''} detected in {titlecase(', '.join(payload['after']['data']['zones']).replace('_', ' '))}"
             message = f"Detected on {camera_name}"
 
         if ended:

frigate/config/auth.py

@@ -20,7 +20,7 @@ class AuthConfig(FrigateBaseModel):
         default=86400, title="Session length for jwt session tokens", ge=60
     )
     refresh_time: int = Field(
-        default=1800,
+        default=43200,
         title="Refresh the session if it is going to expire in this many seconds",
         ge=30,
     )

frigate/config/classification.py

@@ -105,11 +105,6 @@ class CustomClassificationConfig(FrigateBaseModel):
     threshold: float = Field(
         default=0.8, title="Classification score threshold to change the state."
     )
-    save_attempts: int | None = Field(
-        default=None,
-        title="Number of classification attempts to save in the recent classifications tab. If not specified, defaults to 200 for object classification and 100 for state classification.",
-        ge=0,
-    )
     object_config: CustomClassificationObjectConfig | None = Field(default=None)
     state_config: CustomClassificationStateConfig | None = Field(default=None)
 

frigate/config/ui.py

@@ -37,6 +37,9 @@ class UIConfig(FrigateBaseModel):
     time_style: DateTimeStyleEnum = Field(
         default=DateTimeStyleEnum.medium, title="Override UI timeStyle."
     )
+    strftime_fmt: Optional[str] = Field(
+        default=None, title="Override date and time format using strftime syntax."
+    )
     unit_system: UnitSystemEnum = Field(
         default=UnitSystemEnum.metric, title="The unit system to use for measurements."
     )

frigate/data_processing/post/review_descriptions.py

@@ -12,7 +12,6 @@ from typing import Any
 
 import cv2
 from peewee import DoesNotExist
-from titlecase import titlecase
 
 from frigate.comms.embeddings_updater import EmbeddingsRequestEnum
 from frigate.comms.inter_process import InterProcessRequestor
@@ -209,22 +208,10 @@ class ReviewDescriptionProcessor(PostProcessorApi):
             logger.debug(
                 f"Found GenAI Review Summary request for {start_ts} to {end_ts}"
             )
-
-            # Query all review segments with camera and time information
-            segments: list[dict[str, Any]] = [
-                {
-                    "camera": r["camera"].replace("_", " ").title(),
-                    "start_time": r["start_time"],
-                    "end_time": r["end_time"],
-                    "metadata": r["data"]["metadata"],
-                }
+            items: list[dict[str, Any]] = [
+                r["data"]["metadata"]
                 for r in (
-                    ReviewSegment.select(
-                        ReviewSegment.camera,
-                        ReviewSegment.start_time,
-                        ReviewSegment.end_time,
-                        ReviewSegment.data,
-                    )
+                    ReviewSegment.select(ReviewSegment.data)
                     .where(
                         (ReviewSegment.data["metadata"].is_null(False))
                         & (ReviewSegment.start_time < end_ts)
@@ -236,72 +223,21 @@ class ReviewDescriptionProcessor(PostProcessorApi):
                 )
             ]
 
-            if len(segments) == 0:
+            if len(items) == 0:
                 logger.debug("No review items with metadata found during time period")
-                return "No activity was found during this time period."
+                return "No activity was found during this time."
 
-            # Identify primary items (important items that need review)
-            primary_segments = [
-                seg
-                for seg in segments
-                if seg["metadata"].get("potential_threat_level", 0) > 0
-                or seg["metadata"].get("other_concerns")
-            ]
+            important_items = list(
+                filter(
+                    lambda item: item.get("potential_threat_level", 0) > 0
+                    or item.get("other_concerns"),
+                    items,
+                )
+            )
 
-            if not primary_segments:
+            if not important_items:
                 return "No concerns were found during this time period."
 
-            # Build hierarchical structure: each primary event with its contextual items
-            events_with_context = []
-
-            for primary_seg in primary_segments:
-                # Start building the primary event structure
-                primary_item = copy.deepcopy(primary_seg["metadata"])
-                primary_item["camera"] = primary_seg["camera"]
-                primary_item["start_time"] = primary_seg["start_time"]
-                primary_item["end_time"] = primary_seg["end_time"]
-
-                # Find overlapping contextual items from other cameras
-                primary_start = primary_seg["start_time"]
-                primary_end = primary_seg["end_time"]
-                primary_camera = primary_seg["camera"]
-                contextual_items = []
-                seen_contextual_cameras = set()
-
-                for seg in segments:
-                    seg_camera = seg["camera"]
-
-                    if seg_camera == primary_camera:
-                        continue
-
-                    if seg in primary_segments:
-                        continue
-
-                    seg_start = seg["start_time"]
-                    seg_end = seg["end_time"]
-
-                    if seg_start < primary_end and primary_start < seg_end:
-                        # Avoid duplicates if same camera has multiple overlapping segments
-                        if seg_camera not in seen_contextual_cameras:
-                            contextual_item = copy.deepcopy(seg["metadata"])
-                            contextual_item["camera"] = seg_camera
-                            contextual_item["start_time"] = seg_start
-                            contextual_item["end_time"] = seg_end
-                            contextual_items.append(contextual_item)
-                            seen_contextual_cameras.add(seg_camera)
-
-                # Add context array to primary item
-                primary_item["context"] = contextual_items
-                events_with_context.append(primary_item)
-
-            total_context_items = sum(
-                len(event.get("context", [])) for event in events_with_context
-            )
-            logger.debug(
-                f"Summary includes {len(events_with_context)} primary events with "
-                f"{total_context_items} total contextual items"
-            )
-
             if self.config.review.genai.debug_save_thumbnails:
                 Path(
                     os.path.join(CLIPS_DIR, "genai-requests", f"{start_ts}-{end_ts}")
@@ -310,7 +246,7 @@ class ReviewDescriptionProcessor(PostProcessorApi):
             return self.genai_client.generate_review_summary(
                 start_ts,
                 end_ts,
-                events_with_context,
+                important_items,
                 self.config.review.genai.debug_save_thumbnails,
             )
         else:
@@ -519,14 +455,14 @@ def run_analysis(
 
     for i, verified_label in enumerate(final_data["data"]["verified_objects"]):
         object_type = verified_label.replace("-verified", "").replace("_", " ")
-        name = titlecase(sub_labels_list[i].replace("_", " "))
+        name = sub_labels_list[i].replace("_", " ").title()
         unified_objects.append(f"{name} ({object_type})")
 
     for label in objects_list:
         if "-verified" in label:
             continue
         elif label in labelmap_objects:
-            object_type = titlecase(label.replace("_", " "))
+            object_type = label.replace("_", " ").title()
 
             if label in attribute_labels:
                 unified_objects.append(f"{object_type} (delivery/service)")

frigate/data_processing/real_time/custom_classification.py

@@ -99,42 +99,6 @@ class CustomStateClassificationProcessor(RealTimeProcessorApi):
         if self.inference_speed:
             self.inference_speed.update(duration)
 
-    def _should_save_image(
-        self, camera: str, detected_state: str, score: float = 1.0
-    ) -> bool:
-        """
-        Determine if we should save the image for training.
-        Save when:
-        - State is changing or being verified (regardless of score)
-        - Score is less than 100% (even if state matches, useful for training)
-        Don't save when:
-        - State is stable (matches current_state) AND score is 100%
-        """
-        if camera not in self.state_history:
-            # First detection for this camera, save it
-            return True
-
-        verification = self.state_history[camera]
-        current_state = verification.get("current_state")
-        pending_state = verification.get("pending_state")
-
-        # Save if there's a pending state change being verified
-        if pending_state is not None:
-            return True
-
-        # Save if the detected state differs from the current verified state
-        # (state is changing)
-        if current_state is not None and detected_state != current_state:
-            return True
-
-        # If score is less than 100%, save even if state matches
-        # (useful for training to improve confidence)
-        if score < 1.0:
-            return True
-
-        # Don't save if state is stable (detected_state == current_state) AND score is 100%
-        return False
-
     def verify_state_change(self, camera: str, detected_state: str) -> str | None:
         """
         Verify state change requires 3 consecutive identical states before publishing.
@@ -248,22 +212,14 @@ class CustomStateClassificationProcessor(RealTimeProcessorApi):
                 return
 
         if self.interpreter is None:
-            # When interpreter is None, always save (score is 0.0, which is < 1.0)
-            if self._should_save_image(camera, "unknown", 0.0):
-                save_attempts = (
-                    self.model_config.save_attempts
-                    if self.model_config.save_attempts is not None
-                    else 100
-                )
-                write_classification_attempt(
-                    self.train_dir,
-                    cv2.cvtColor(frame, cv2.COLOR_RGB2BGR),
-                    "none-none",
-                    now,
-                    "unknown",
-                    0.0,
-                    max_files=save_attempts,
-                )
+            write_classification_attempt(
+                self.train_dir,
+                cv2.cvtColor(frame, cv2.COLOR_RGB2BGR),
+                "none-none",
+                now,
+                "unknown",
+                0.0,
+            )
             return
 
         input = np.expand_dims(resized_frame, axis=0)
@@ -280,23 +236,14 @@ class CustomStateClassificationProcessor(RealTimeProcessorApi):
         score = round(probs[best_id], 2)
         self.__update_metrics(datetime.datetime.now().timestamp() - now)
 
-        detected_state = self.labelmap[best_id]
-
-        if self._should_save_image(camera, detected_state, score):
-            save_attempts = (
-                self.model_config.save_attempts
-                if self.model_config.save_attempts is not None
-                else 100
-            )
-            write_classification_attempt(
-                self.train_dir,
-                cv2.cvtColor(frame, cv2.COLOR_RGB2BGR),
-                "none-none",
-                now,
-                detected_state,
-                score,
-                max_files=save_attempts,
-            )
+        write_classification_attempt(
+            self.train_dir,
+            cv2.cvtColor(frame, cv2.COLOR_RGB2BGR),
+            "none-none",
+            now,
+            self.labelmap[best_id],
+            score,
+        )
 
         if score < self.model_config.threshold:
             logger.debug(
@@ -304,6 +251,7 @@ class CustomStateClassificationProcessor(RealTimeProcessorApi):
             )
             return
 
+        detected_state = self.labelmap[best_id]
         verified_state = self.verify_state_change(camera, detected_state)
 
         if verified_state is not None:
@@ -457,6 +405,9 @@ class CustomObjectClassificationProcessor(RealTimeProcessorApi):
         if obj_data.get("end_time") is not None:
             return
 
+        if obj_data.get("stationary"):
+            return
+
         object_id = obj_data["id"]
 
         if (
@@ -494,11 +445,6 @@ class CustomObjectClassificationProcessor(RealTimeProcessorApi):
                 return
 
         if self.interpreter is None:
-            save_attempts = (
-                self.model_config.save_attempts
-                if self.model_config.save_attempts is not None
-                else 200
-            )
             write_classification_attempt(
                 self.train_dir,
                 cv2.cvtColor(crop, cv2.COLOR_RGB2BGR),
@@ -506,7 +452,6 @@ class CustomObjectClassificationProcessor(RealTimeProcessorApi):
                 now,
                 "unknown",
                 0.0,
-                max_files=save_attempts,
             )
             return
 
@@ -524,11 +469,6 @@ class CustomObjectClassificationProcessor(RealTimeProcessorApi):
         score = round(probs[best_id], 2)
         self.__update_metrics(datetime.datetime.now().timestamp() - now)
 
-        save_attempts = (
-            self.model_config.save_attempts
-            if self.model_config.save_attempts is not None
-            else 200
-        )
         write_classification_attempt(
             self.train_dir,
             cv2.cvtColor(crop, cv2.COLOR_RGB2BGR),
@@ -536,7 +476,7 @@ class CustomObjectClassificationProcessor(RealTimeProcessorApi):
             now,
             self.labelmap[best_id],
             score,
-            max_files=save_attempts,
+            max_files=200,
         )
 
         if score < self.model_config.threshold:
@@ -639,14 +579,14 @@ def write_classification_attempt(
     os.makedirs(folder, exist_ok=True)
     cv2.imwrite(file, frame)
 
+    files = sorted(
+        filter(lambda f: (f.endswith(".webp")), os.listdir(folder)),
+        key=lambda f: os.path.getctime(os.path.join(folder, f)),
+        reverse=True,
+    )
+
     # delete oldest face image if maximum is reached
     try:
-        files = sorted(
-            filter(lambda f: (f.endswith(".webp")), os.listdir(folder)),
-            key=lambda f: os.path.getctime(os.path.join(folder, f)),
-            reverse=True,
-        )
-
         if len(files) > max_files:
             os.unlink(os.path.join(folder, files[-1]))
     except FileNotFoundError:

frigate/detectors/plugins/edgetpu_tfl.py

@@ -1,20 +1,19 @@
 import logging
-import math
 import os
 
-import cv2
 import numpy as np
 from pydantic import Field
 from typing_extensions import Literal
 
 from frigate.detectors.detection_api import DetectionApi
-from frigate.detectors.detector_config import BaseDetectorConfig, ModelTypeEnum
+from frigate.detectors.detector_config import BaseDetectorConfig
 
 try:
     from tflite_runtime.interpreter import Interpreter, load_delegate
 except ModuleNotFoundError:
     from tensorflow.lite.python.interpreter import Interpreter, load_delegate
 
+
 logger = logging.getLogger(__name__)
 
 DETECTOR_KEY = "edgetpu"
@@ -27,10 +26,6 @@ class EdgeTpuDetectorConfig(BaseDetectorConfig):
 
 class EdgeTpuTfl(DetectionApi):
     type_key = DETECTOR_KEY
-    supported_models = [
-        ModelTypeEnum.ssd,
-        ModelTypeEnum.yologeneric,
-    ]
 
     def __init__(self, detector_config: EdgeTpuDetectorConfig):
         device_config = {}
@@ -68,294 +63,31 @@ class EdgeTpuTfl(DetectionApi):
 
         self.tensor_input_details = self.interpreter.get_input_details()
         self.tensor_output_details = self.interpreter.get_output_details()
-        self.model_width = detector_config.model.width
-        self.model_height = detector_config.model.height
-
-        self.min_score = 0.4
-        self.max_detections = 20
-
         self.model_type = detector_config.model.model_type
-        self.model_requires_int8 = self.tensor_input_details[0]["dtype"] == np.int8
-
-        if self.model_type == ModelTypeEnum.yologeneric:
-            logger.debug("Using YOLO preprocessing/postprocessing")
-
-            if len(self.tensor_output_details) not in [2, 3]:
-                logger.error(
-                    f"Invalid count of output tensors in YOLO model. Found {len(self.tensor_output_details)}, expecting 2 or 3."
-                )
-                raise
-
-            self.reg_max = 16  # = 64 dfl_channels // 4 # YOLO standard
-            self.min_logit_value = np.log(
-                self.min_score / (1 - self.min_score)
-            )  # for filtering
-            self._generate_anchors_and_strides()  # decode bounding box DFL
-            self.project = np.arange(
-                self.reg_max, dtype=np.float32
-            )  # for decoding bounding box DFL information
-
-            # Determine YOLO tensor indices and quantization scales for
-            # boxes and class_scores the tensor ordering and names are
-            # not reliable, so use tensor shape to detect which tensor
-            # holds boxes or class scores.
-            # The tensors have shapes (B, N, C)
-            # where N is the number of candidates (=2100 for 320x320)
-            # this may guess wrong if the number of classes is exactly 64
-            output_boxes_index = None
-            output_classes_index = None
-            for i, x in enumerate(self.tensor_output_details):
-                # the nominal index seems to start at 1 instead of 0
-                if len(x["shape"]) == 3 and x["shape"][2] == 64:
-                    output_boxes_index = i
-                elif len(x["shape"]) == 3 and x["shape"][2] > 1:
-                    # require the number of classes to be more than 1
-                    # to differentiate from (not used) max score tensor
-                    output_classes_index = i
-            if output_boxes_index is None or output_classes_index is None:
-                logger.warning("Unrecognized model output, unexpected tensor shapes.")
-                output_classes_index = (
-                    0
-                    if (output_boxes_index is None or output_classes_index == 1)
-                    else 1
-                )  # 0 is default guess
-                output_boxes_index = 1 if (output_boxes_index == 0) else 0
-
-            scores_details = self.tensor_output_details[output_classes_index]
-            self.scores_tensor_index = scores_details["index"]
-            self.scores_scale, self.scores_zero_point = scores_details["quantization"]
-            # calculate the quantized version of the min_score
-            self.min_score_quantized = int(
-                (self.min_logit_value / self.scores_scale) + self.scores_zero_point
-            )
-            self.logit_shift_to_positive_values = (
-                max(0, math.ceil((128 + self.scores_zero_point) * self.scores_scale))
-                + 1
-            )  # round up
-
-            boxes_details = self.tensor_output_details[output_boxes_index]
-            self.boxes_tensor_index = boxes_details["index"]
-            self.boxes_scale, self.boxes_zero_point = boxes_details["quantization"]
-
-        elif self.model_type == ModelTypeEnum.ssd:
-            logger.debug("Using SSD preprocessing/postprocessing")
-
-            # SSD model indices (4 outputs: boxes, class_ids, scores, count)
-            for x in self.tensor_output_details:
-                if len(x["shape"]) == 3:
-                    self.output_boxes_index = x["index"]
-                elif len(x["shape"]) == 1:
-                    self.output_count_index = x["index"]
-
-            self.output_class_ids_index = None
-            self.output_class_scores_index = None
-
-        else:
-            raise Exception(
-                f"{self.model_type} is currently not supported for edgetpu. See the docs for more info on supported models."
-            )
-
-    def _generate_anchors_and_strides(self):
-        # for decoding the bounding box DFL information into xy coordinates
-        all_anchors = []
-        all_strides = []
-        strides = (8, 16, 32)  # YOLO's small, medium, large detection heads
-
-        for stride in strides:
-            feat_h, feat_w = self.model_height // stride, self.model_width // stride
-
-            grid_y, grid_x = np.meshgrid(
-                np.arange(feat_h, dtype=np.float32),
-                np.arange(feat_w, dtype=np.float32),
-                indexing="ij",
-            )
-
-            grid_coords = np.stack((grid_x.flatten(), grid_y.flatten()), axis=1)
-            anchor_points = grid_coords + 0.5
-
-            all_anchors.append(anchor_points)
-            all_strides.append(np.full((feat_h * feat_w, 1), stride, dtype=np.float32))
-
-        self.anchors = np.concatenate(all_anchors, axis=0)
-        self.anchor_strides = np.concatenate(all_strides, axis=0)
-
-    def determine_indexes_for_non_yolo_models(self):
-        """Legacy method for SSD models."""
-        if (
-            self.output_class_ids_index is None
-            or self.output_class_scores_index is None
-        ):
-            for i in range(4):
-                index = self.tensor_output_details[i]["index"]
-                if (
-                    index != self.output_boxes_index
-                    and index != self.output_count_index
-                ):
-                    if (
-                        np.mod(np.float32(self.interpreter.tensor(index)()[0][0]), 1)
-                        == 0.0
-                    ):
-                        self.output_class_ids_index = index
-                    else:
-                        self.output_scores_index = index
-
-    def pre_process(self, tensor_input):
-        if self.model_requires_int8:
-            tensor_input = np.bitwise_xor(tensor_input, 128).view(
-                np.int8
-            )  # shift by -128
-        return tensor_input
 
     def detect_raw(self, tensor_input):
-        tensor_input = self.pre_process(tensor_input)
-
         self.interpreter.set_tensor(self.tensor_input_details[0]["index"], tensor_input)
         self.interpreter.invoke()
 
-        if self.model_type == ModelTypeEnum.yologeneric:
-            # Multi-tensor YOLO model with (non-standard B(H*W)C output format).
-            # (the comments indicate the shape of tensors,
-            # using "2100" as the anchor count (for image size of 320x320),
-            # "NC" as number of classes,
-            # "N" as the count that survive after min-score filtering)
-            # TENSOR A) class scores (1, 2100, NC) with logit values
-            # TENSOR B) box coordinates (1, 2100, 64) encoded as dfl scores
-            # Recommend that the model clamp the logit values in tensor (A)
-            # to the range [-4,+4] to preserve precision from [2%,98%]
-            # and because NMS requires the min_score parameter to be >= 0
+        boxes = self.interpreter.tensor(self.tensor_output_details[0]["index"])()[0]
+        class_ids = self.interpreter.tensor(self.tensor_output_details[1]["index"])()[0]
+        scores = self.interpreter.tensor(self.tensor_output_details[2]["index"])()[0]
+        count = int(
+            self.interpreter.tensor(self.tensor_output_details[3]["index"])()[0]
+        )
 
-            # don't dequantize scores data yet, wait until the low-confidence
-            # candidates are filtered out from the overall result set.
-            # This reduces the work and makes post-processing faster.
-            # this method works with raw quantized numbers when possible,
-            # which relies on the value of the scale factor to be >0.
-            # This speeds up max and argmax operations.
-            # Get max confidence for each detection and create the mask
-            detections = np.zeros(
-                (self.max_detections, 6), np.float32
-            )  # initialize zero results
-            scores_output_quantized = self.interpreter.get_tensor(
-                self.scores_tensor_index
-            )[0]  # (2100, NC)
-            max_scores_quantized = np.max(scores_output_quantized, axis=1)  # (2100,)
-            mask = max_scores_quantized >= self.min_score_quantized  # (2100,)
+        detections = np.zeros((20, 6), np.float32)
 
-            if not np.any(mask):
-                return detections  # empty results
-
-            max_scores_filtered_shiftedpositive = (
-                (max_scores_quantized[mask] - self.scores_zero_point)
-                * self.scores_scale
-            ) + self.logit_shift_to_positive_values  # (N,1) shifted logit values
-            scores_output_quantized_filtered = scores_output_quantized[mask]
-
-            # dequantize boxes. NMS needs them to be in float format
-            # remove candidates with probabilities < threshold
-            boxes_output_quantized_filtered = (
-                self.interpreter.get_tensor(self.boxes_tensor_index)[0]
-            )[mask]  # (N, 64)
-            boxes_output_filtered = (
-                boxes_output_quantized_filtered.astype(np.float32)
-                - self.boxes_zero_point
-            ) * self.boxes_scale
-
-            # 2. Decode DFL to distances (ltrb)
-            dfl_distributions = boxes_output_filtered.reshape(
-                -1, 4, self.reg_max
-            )  # (N, 4, 16)
-
-            # Softmax over the 16 bins
-            dfl_max = np.max(dfl_distributions, axis=2, keepdims=True)
-            dfl_exp = np.exp(dfl_distributions - dfl_max)
-            dfl_probs = dfl_exp / np.sum(dfl_exp, axis=2, keepdims=True)  # (N, 4, 16)
-
-            # Weighted sum: (N, 4, 16) * (16,) -> (N, 4)
-            distances = np.einsum("pcr,r->pc", dfl_probs, self.project)
-
-            # Calculate box corners in pixel coordinates
-            anchors_filtered = self.anchors[mask]
-            anchor_strides_filtered = self.anchor_strides[mask]
-            x1y1 = (
-                anchors_filtered - distances[:, [0, 1]]
-            ) * anchor_strides_filtered  # (N, 2)
-            x2y2 = (
-                anchors_filtered + distances[:, [2, 3]]
-            ) * anchor_strides_filtered  # (N, 2)
-            boxes_filtered_decoded = np.concatenate((x1y1, x2y2), axis=-1)  # (N, 4)
-
-            # 9. Apply NMS. Use logit scores here to defer sigmoid()
-            # until after filtering out redundant boxes
-            # Shift the logit scores to be non-negative (required by cv2)
-            indices = cv2.dnn.NMSBoxes(
-                bboxes=boxes_filtered_decoded,
-                scores=max_scores_filtered_shiftedpositive,
-                score_threshold=(
-                    self.min_logit_value + self.logit_shift_to_positive_values
-                ),
-                nms_threshold=0.4,  # should this be a model config setting?
-            )
-            num_detections = len(indices)
-            if num_detections == 0:
-                return detections  # empty results
-
-            nms_indices = np.array(indices, dtype=np.int32).ravel()  # or .flatten()
-            if num_detections > self.max_detections:
-                nms_indices = nms_indices[: self.max_detections]
-                num_detections = self.max_detections
-            kept_logits_quantized = scores_output_quantized_filtered[nms_indices]
-            class_ids_post_nms = np.argmax(kept_logits_quantized, axis=1)
-
-            # Extract the final boxes and scores using fancy indexing
-            final_boxes = boxes_filtered_decoded[nms_indices]
-            final_scores_logits = (
-                max_scores_filtered_shiftedpositive[nms_indices]
-                - self.logit_shift_to_positive_values
-            )  # Unshifted logits
-
-            # Detections array format: [class_id, score, ymin, xmin, ymax, xmax]
-            detections[:num_detections, 0] = class_ids_post_nms
-            detections[:num_detections, 1] = 1.0 / (
-                1.0 + np.exp(-final_scores_logits)
-            )  # sigmoid
-            detections[:num_detections, 2] = final_boxes[:, 1] / self.model_height
-            detections[:num_detections, 3] = final_boxes[:, 0] / self.model_width
-            detections[:num_detections, 4] = final_boxes[:, 3] / self.model_height
-            detections[:num_detections, 5] = final_boxes[:, 2] / self.model_width
-            return detections
-
-        elif self.model_type == ModelTypeEnum.ssd:
-            self.determine_indexes_for_non_yolo_models()
-            boxes = self.interpreter.tensor(self.tensor_output_details[0]["index"])()[0]
-            class_ids = self.interpreter.tensor(
-                self.tensor_output_details[1]["index"]
-            )()[0]
-            scores = self.interpreter.tensor(self.tensor_output_details[2]["index"])()[
-                0
+        for i in range(count):
+            if scores[i] < 0.4 or i == 20:
+                break
+            detections[i] = [
+                class_ids[i],
+                float(scores[i]),
+                boxes[i][0],
+                boxes[i][1],
+                boxes[i][2],
+                boxes[i][3],
             ]
-            count = int(
-                self.interpreter.tensor(self.tensor_output_details[3]["index"])()[0]
-            )
 
-            detections = np.zeros((self.max_detections, 6), np.float32)
-
-            for i in range(count):
-                if scores[i] < self.min_score:
-                    break
-                if i == self.max_detections:
-                    logger.debug(f"Too many detections ({count})!")
-                    break
-                detections[i] = [
-                    class_ids[i],
-                    float(scores[i]),
-                    boxes[i][0],
-                    boxes[i][1],
-                    boxes[i][2],
-                    boxes[i][3],
-                ]
-
-            return detections
-
-        else:
-            raise Exception(
-                f"{self.model_type} is currently not supported for edgetpu. See the docs for more info on supported models."
-            )
+        return detections

frigate/detectors/plugins/memryx.py

@@ -2,6 +2,7 @@ import glob
 import logging
 import os
 import shutil
+import time
 import urllib.request
 import zipfile
 from queue import Queue
@@ -54,9 +55,6 @@ class MemryXDetector(DetectionApi):
             )
             return
 
-        # Initialize stop_event as None, will be set later by set_stop_event()
-        self.stop_event = None
-
         model_cfg = getattr(detector_config, "model", None)
 
         # Check if model_type was explicitly set by the user
@@ -365,43 +363,26 @@ class MemryXDetector(DetectionApi):
     def process_input(self):
         """Input callback function: wait for frames in the input queue, preprocess, and send to MX3 (return)"""
         while True:
-            # Check if shutdown is requested
-            if self.stop_event and self.stop_event.is_set():
-                logger.debug("[process_input] Stop event detected, returning None")
-                return None
             try:
-                # Wait for a frame from the queue with timeout to check stop_event periodically
-                frame = self.capture_queue.get(block=True, timeout=0.5)
+                # Wait for a frame from the queue (blocking call)
+                frame = self.capture_queue.get(
+                    block=True
+                )  # Blocks until data is available
 
                 return frame
 
             except Exception as e:
-                # Silently handle queue.Empty timeouts (expected during normal operation)
-                # Log any other unexpected exceptions
-                if "Empty" not in str(type(e).__name__):
-                    logger.warning(f"[process_input] Unexpected error: {e}")
-                # Loop continues and will check stop_event at the top
+                logger.info(f"[process_input] Error processing input: {e}")
+                time.sleep(0.1)  # Prevent busy waiting in case of error
 
     def receive_output(self):
         """Retrieve processed results from MemryX output queue + a copy of the original frame"""
-        try:
-            # Get connection ID with timeout
-            connection_id = self.capture_id_queue.get(
-                block=True, timeout=1.0
-            )  # Get the corresponding connection ID
-            detections = self.output_queue.get()  # Get detections from MemryX
+        connection_id = (
+            self.capture_id_queue.get()
+        )  # Get the corresponding connection ID
+        detections = self.output_queue.get()  # Get detections from MemryX
 
-            return connection_id, detections
-
-        except Exception as e:
-            # On timeout or stop event, return None
-            if self.stop_event and self.stop_event.is_set():
-                logger.debug("[receive_output] Stop event detected, exiting")
-            # Silently handle queue.Empty timeouts, they're expected during normal operation
-            elif "Empty" not in str(type(e).__name__):
-                logger.warning(f"[receive_output] Error receiving output: {e}")
-
-            return None, None
+        return connection_id, detections
 
     def post_process_yolonas(self, output):
         predictions = output[0]
@@ -850,19 +831,6 @@ class MemryXDetector(DetectionApi):
                 f"{self.memx_model_type} is currently not supported for memryx. See the docs for more info on supported models."
             )
 
-    def set_stop_event(self, stop_event):
-        """Set the stop event for graceful shutdown."""
-        self.stop_event = stop_event
-
-    def shutdown(self):
-        """Gracefully shutdown the MemryX accelerator"""
-        try:
-            if hasattr(self, "accl") and self.accl is not None:
-                self.accl.shutdown()
-                logger.info("MemryX accelerator shutdown complete")
-        except Exception as e:
-            logger.error(f"Error during MemryX shutdown: {e}")
-
     def detect_raw(self, tensor_input: np.ndarray):
         """Removed synchronous detect_raw() function so that we only use async"""
         return 0

frigate/embeddings/embeddings.py

@@ -12,6 +12,9 @@ from peewee import DoesNotExist, IntegrityError
 from PIL import Image
 from playhouse.shortcuts import model_to_dict
 
+from frigate.comms.embeddings_updater import (
+    EmbeddingsRequestEnum,
+)
 from frigate.comms.inter_process import InterProcessRequestor
 from frigate.config import FrigateConfig
 from frigate.config.classification import SemanticSearchModelEnum
@@ -492,49 +495,44 @@ class Embeddings:
                         or thumbnail_missing
                     ):
                         existing_trigger.embedding = self._calculate_trigger_embedding(
-                            trigger, trigger_name, camera.name
+                            trigger
                         )
                         needs_embedding_update = True
 
                     if needs_embedding_update:
                         existing_trigger.save()
-                    continue
                 else:
                     # Create new trigger
                     try:
-                        # For thumbnail triggers, validate the event exists
-                        if trigger.type == "thumbnail":
-                            try:
-                                event: Event = Event.get(Event.id == trigger.data)
-                            except DoesNotExist:
-                                logger.warning(
-                                    f"Event ID {trigger.data} for trigger {trigger_name} does not exist."
-                                )
-                                continue
-
-                            # Skip the event if not an object
-                            if event.data.get("type") != "object":
-                                logger.warning(
-                                    f"Event ID {trigger.data} for trigger {trigger_name} is not a tracked object."
-                                )
-                                continue
-
-                            thumbnail = get_event_thumbnail_bytes(event)
-
-                            if not thumbnail:
-                                logger.warning(
-                                    f"Unable to retrieve thumbnail for event ID {trigger.data} for {trigger_name}."
-                                )
-                                continue
-
-                            self.write_trigger_thumbnail(
-                                camera.name, trigger.data, thumbnail
+                        try:
+                            event: Event = Event.get(Event.id == trigger.data)
+                        except DoesNotExist:
+                            logger.warning(
+                                f"Event ID {trigger.data} for trigger {trigger_name} does not exist."
                             )
+                            continue
+
+                        # Skip the event if not an object
+                        if event.data.get("type") != "object":
+                            logger.warning(
+                                f"Event ID {trigger.data} for trigger {trigger_name} is not a tracked object."
+                            )
+                            continue
+
+                        thumbnail = get_event_thumbnail_bytes(event)
+
+                        if not thumbnail:
+                            logger.warning(
+                                f"Unable to retrieve thumbnail for event ID {trigger.data} for {trigger_name}."
+                            )
+                            continue
+
+                        self.write_trigger_thumbnail(
+                            camera.name, trigger.data, thumbnail
+                        )
 
                         # Calculate embedding for new trigger
-                        embedding = self._calculate_trigger_embedding(
-                            trigger, trigger_name, camera.name
-                        )
+                        embedding = self._calculate_trigger_embedding(trigger)
 
                         Trigger.create(
                             camera=camera.name,
@@ -560,11 +558,7 @@ class Embeddings:
                     Trigger.camera == camera.name, Trigger.name.in_(triggers_to_remove)
                 ).execute()
                 for trigger_name in triggers_to_remove:
-                    # Only remove thumbnail files for thumbnail triggers
-                    if existing_triggers[trigger_name].type == "thumbnail":
-                        self.remove_trigger_thumbnail(
-                            camera.name, existing_triggers[trigger_name].data
-                        )
+                    self.remove_trigger_thumbnail(camera.name, trigger_name)
 
     def write_trigger_thumbnail(
         self, camera: str, event_id: str, thumbnail: bytes
@@ -594,13 +588,14 @@ class Embeddings:
                 f"Failed to delete thumbnail for trigger with data {event_id} in {camera}: {e}"
             )
 
-    def _calculate_trigger_embedding(
-        self, trigger, trigger_name: str, camera_name: str
-    ) -> bytes:
+    def _calculate_trigger_embedding(self, trigger) -> bytes:
         """Calculate embedding for a trigger based on its type and data."""
         if trigger.type == "description":
-            logger.debug(f"Generating embedding for trigger description {trigger_name}")
-            embedding = self.embed_description(None, trigger.data, upsert=False)
+            logger.debug(f"Generating embedding for trigger description {trigger.name}")
+            embedding = self.requestor.send_data(
+                EmbeddingsRequestEnum.embed_description.value,
+                {"id": None, "description": trigger.data, "upsert": False},
+            )
             return embedding.astype(np.float32).tobytes()
 
         elif trigger.type == "thumbnail":
@@ -620,21 +615,28 @@ class Embeddings:
 
                 try:
                     with open(
-                        os.path.join(TRIGGER_DIR, camera_name, f"{trigger.data}.webp"),
+                        os.path.join(
+                            TRIGGER_DIR, trigger.camera, f"{trigger.data}.webp"
+                        ),
                         "rb",
                     ) as f:
                         thumbnail = f.read()
                 except Exception as e:
                     logger.error(
-                        f"Failed to read thumbnail for trigger {trigger_name} with ID {trigger.data}: {e}"
+                        f"Failed to read thumbnail for trigger {trigger.name} with ID {trigger.data}: {e}"
                     )
                     return b""
 
                 logger.debug(
-                    f"Generating embedding for trigger thumbnail {trigger_name} with ID {trigger.data}"
+                    f"Generating embedding for trigger thumbnail {trigger.name} with ID {trigger.data}"
                 )
-                embedding = self.embed_thumbnail(
-                    str(trigger.data), thumbnail, upsert=False
+                embedding = self.requestor.send_data(
+                    EmbeddingsRequestEnum.embed_thumbnail.value,
+                    {
+                        "id": str(trigger.data),
+                        "thumbnail": str(thumbnail),
+                        "upsert": False,
+                    },
                 )
                 return embedding.astype(np.float32).tobytes()
 

frigate/genai/__init__.py

@@ -177,60 +177,50 @@ Each line represents a detection state, not necessarily unique individuals. Pare
         self,
         start_ts: float,
         end_ts: float,
-        events: list[dict[str, Any]],
+        segments: list[dict[str, Any]],
         debug_save: bool,
     ) -> str | None:
         """Generate a summary of review item descriptions over a period of time."""
         time_range = f"{datetime.datetime.fromtimestamp(start_ts).strftime('%B %d, %Y at %I:%M %p')} to {datetime.datetime.fromtimestamp(end_ts).strftime('%B %d, %Y at %I:%M %p')}"
         timeline_summary_prompt = f"""
-You are a security officer writing a concise security report.
+You are a security officer.
+Time range: {time_range}.
+Input: JSON list with "title", "scene", "confidence", "potential_threat_level" (1-2), "other_concerns".
 
-Time range: {time_range}
+Task: Write a concise, human-presentable security report in markdown format.
 
-Input format: Each event is a JSON object with:
-- "title", "scene", "confidence", "potential_threat_level" (0-2), "other_concerns", "camera", "time", "start_time", "end_time"
-- "context": array of related events from other cameras that occurred during overlapping time periods
+Rules for the report:
 
-Report Structure - Use this EXACT format:
+- Title & overview
+  - Start with:
+    # Security Summary - {time_range}
+  - Write a 1-2 sentence situational overview capturing the general pattern of the period.
 
-# Security Summary - {time_range}
+- Event details
+  - Present events in chronological order as a bullet list.
+  - **If multiple events occur within the same minute or overlapping time range, COMBINE them into a single bullet.**
+    - Summarize the distinct activities as sub-points under the shared timestamp.
+  - If no timestamp is given, preserve order but label as “Time not specified.”
+  - Use bold timestamps for clarity.
+  - Group bullets under subheadings when multiple events fall into the same category (e.g., Vehicle Activity, Porch Activity, Unusual Behavior).
 
-## Overview
-[Write 1-2 sentences summarizing the overall activity pattern during this period.]
+- Threat levels
+  - Always show (threat level: X) for each event.
+  - If multiple events at the same time share the same threat level, only state it once.
 
----
+- Final assessment
+  - End with a Final Assessment section.
+  - If all events are threat level 1 with no escalation:
+    Final assessment: Only normal residential activity observed during this period.
+  - If threat level 2+ events are present, clearly summarize them as Potential concerns requiring review.
 
-## Timeline
-
-[Group events by time periods (e.g., "Morning (6:00 AM - 12:00 PM)", "Afternoon (12:00 PM - 5:00 PM)", "Evening (5:00 PM - 9:00 PM)", "Night (9:00 PM - 6:00 AM)"). Use appropriate time blocks based on when events occurred.]
-
-### [Time Block Name]
-
-**HH:MM AM/PM** | [Camera Name] | [Threat Level Indicator]
-- [Event title]: [Clear description incorporating contextual information from the "context" array]
-- Context: [If context array has items, mention them here, e.g., "Delivery truck present on Front Driveway Cam (HH:MM AM/PM)"]
-- Assessment: [Brief assessment incorporating context - if context explains the event, note it here]
-
-[Repeat for each event in chronological order within the time block]
-
----
-
-## Summary
-[One sentence summarizing the period. If all events are normal/explained: "Routine activity observed." If review needed: "Some activity requires review but no security concerns." If security concerns: "Security concerns requiring immediate attention."]
-
-Guidelines:
-- List ALL events in chronological order, grouped by time blocks
-- Threat level indicators: ✓ Normal, ⚠️ Needs review, 🔴 Security concern
-- Integrate contextual information naturally - use the "context" array to enrich each event's description
-- If context explains the event (e.g., delivery truck explains person at door), describe it accordingly (e.g., "delivery person" not "unidentified person")
-- Be concise but informative - focus on what happened and what it means
-- If contextual information makes an event clearly normal, reflect that in your assessment
-- Only create time blocks that have events - don't create empty sections
+- Conciseness
+  - Do not repeat benign clothing/appearance details unless they distinguish individuals.
+  - Summarize similar routine events instead of restating full scene descriptions.
 """
 
-        timeline_summary_prompt += "\n\nEvents:\n"
-        for event in events:
-            timeline_summary_prompt += f"\n{event}\n"
+        for item in segments:
+            timeline_summary_prompt += f"\n{item}"
 
         if debug_save:
             with open(

frigate/models.py

@@ -133,7 +133,6 @@ class User(Model):
         default="admin",
     )
     password_hash = CharField(null=False, max_length=120)
-    password_changed_at = DateTimeField(null=True)
     notification_tokens = JSONField()
 
     @classmethod

frigate/motion/improved_motion.py

@@ -239,12 +239,6 @@ class ImprovedMotionDetector(MotionDetector):
         )
         self.mask = np.where(resized_mask == [0])
 
-        # Reset motion detection state when mask changes
-        # so motion detection can quickly recalibrate with the new mask
-        self.avg_frame = np.zeros(self.motion_frame_size, np.float32)
-        self.calibrating = True
-        self.motion_frame_count = 0
-
     def stop(self) -> None:
         """stop the motion detector."""
         pass

frigate/object_detection/base.py

@@ -43,7 +43,6 @@ class BaseLocalDetector(ObjectDetector):
         self,
         detector_config: BaseDetectorConfig = None,
         labels: str = None,
-        stop_event: MpEvent = None,
     ):
         self.fps = EventsPerSecond()
         if labels is None:
@@ -61,10 +60,6 @@ class BaseLocalDetector(ObjectDetector):
 
         self.detect_api = create_detector(detector_config)
 
-        # If the detector supports stop_event, pass it
-        if hasattr(self.detect_api, "set_stop_event") and stop_event:
-            self.detect_api.set_stop_event(stop_event)
-
     def _transform_input(self, tensor_input: np.ndarray) -> np.ndarray:
         if self.input_transform:
             tensor_input = np.transpose(tensor_input, self.input_transform)
@@ -245,10 +240,6 @@ class AsyncDetectorRunner(FrigateProcess):
         while not self.stop_event.is_set():
             connection_id, detections = self._detector.async_receive_output()
 
-            # Handle timeout case (queue.Empty) - just continue
-            if connection_id is None:
-                continue
-
             if not self.send_times:
                 # guard; shouldn't happen if send/recv are balanced
                 continue
@@ -275,38 +266,21 @@ class AsyncDetectorRunner(FrigateProcess):
 
         self._frame_manager = SharedMemoryFrameManager()
         self._publisher = ObjectDetectorPublisher()
-        self._detector = AsyncLocalObjectDetector(
-            detector_config=self.detector_config, stop_event=self.stop_event
-        )
+        self._detector = AsyncLocalObjectDetector(detector_config=self.detector_config)
 
         for name in self.cameras:
             self.create_output_shm(name)
 
-        t_detect = threading.Thread(target=self._detect_worker, daemon=False)
-        t_result = threading.Thread(target=self._result_worker, daemon=False)
+        t_detect = threading.Thread(target=self._detect_worker, daemon=True)
+        t_result = threading.Thread(target=self._result_worker, daemon=True)
         t_detect.start()
         t_result.start()
 
-        try:
-            while not self.stop_event.is_set():
-                time.sleep(0.5)
+        while not self.stop_event.is_set():
+            time.sleep(0.5)
 
-            logger.info(
-                "Stop event detected, waiting for detector threads to finish..."
-            )
-
-            # Wait for threads to finish processing
-            t_detect.join(timeout=5)
-            t_result.join(timeout=5)
-
-            # Shutdown the AsyncDetector
-            self._detector.detect_api.shutdown()
-
-            self._publisher.stop()
-        except Exception as e:
-            logger.error(f"Error during async detector shutdown: {e}")
-        finally:
-            logger.info("Exited Async detection process...")
+        self._publisher.stop()
+        logger.info("Exited async detection process...")
 
 
 class ObjectDetectProcess:
@@ -334,7 +308,7 @@ class ObjectDetectProcess:
         # if the process has already exited on its own, just return
         if self.detect_process and self.detect_process.exitcode:
             return
-
+        self.detect_process.terminate()
         logging.info("Waiting for detection process to exit gracefully...")
         self.detect_process.join(timeout=30)
         if self.detect_process.exitcode is None:

frigate/ptz/onvif.py

@@ -95,21 +95,12 @@ class OnvifController:
 
         cam = self.camera_configs[cam_name]
         try:
-            user = cam.onvif.user
-            password = cam.onvif.password
-
-            if user is not None and isinstance(user, bytes):
-                user = user.decode("utf-8")
-
-            if password is not None and isinstance(password, bytes):
-                password = password.decode("utf-8")
-
             self.cams[cam_name] = {
                 "onvif": ONVIFCamera(
                     cam.onvif.host,
                     cam.onvif.port,
-                    user,
-                    password,
+                    cam.onvif.user,
+                    cam.onvif.password,
                     wsdl_dir=str(Path(find_spec("onvif").origin).parent / "wsdl"),
                     adjust_time=cam.onvif.ignore_time_mismatch,
                     encrypt=not cam.onvif.tls_insecure,
@@ -199,11 +190,7 @@ class OnvifController:
         ptz: ONVIFService = await onvif.create_ptz_service()
         self.cams[camera_name]["ptz"] = ptz
 
-        try:
-            imaging: ONVIFService = await onvif.create_imaging_service()
-        except (Fault, ONVIFError, TransportError, Exception) as e:
-            logger.debug(f"Imaging service not supported for {camera_name}: {e}")
-            imaging = None
+        imaging: ONVIFService = await onvif.create_imaging_service()
         self.cams[camera_name]["imaging"] = imaging
         try:
             video_sources = await media.GetVideoSources()
@@ -334,15 +321,9 @@ class OnvifController:
             presets = []
 
         for preset in presets:
-            # Ensure preset name is a Unicode string and handle UTF-8 characters correctly
-            preset_name = getattr(preset, "Name") or f"preset {preset['token']}"
-
-            if isinstance(preset_name, bytes):
-                preset_name = preset_name.decode("utf-8")
-
-            # Convert to lowercase while preserving UTF-8 characters
-            preset_name_lower = preset_name.lower()
-            self.cams[camera_name]["presets"][preset_name_lower] = preset["token"]
+            self.cams[camera_name]["presets"][
+                (getattr(preset, "Name") or f"preset {preset['token']}").lower()
+            ] = preset["token"]
 
         # get list of supported features
         supported_features = []
@@ -400,10 +381,7 @@ class OnvifController:
                             f"Disabling autotracking zooming for {camera_name}: Absolute zoom not supported. Exception: {e}"
                         )
 
-        if (
-            self.cams[camera_name]["video_source_token"] is not None
-            and imaging is not None
-        ):
+        if self.cams[camera_name]["video_source_token"] is not None:
             try:
                 imaging_capabilities = await imaging.GetImagingSettings(
                     {"VideoSourceToken": self.cams[camera_name]["video_source_token"]}
@@ -443,7 +421,6 @@ class OnvifController:
         if (
             "focus" in self.cams[camera_name]["features"]
             and self.cams[camera_name]["video_source_token"]
-            and self.cams[camera_name]["imaging"] is not None
         ):
             try:
                 stop_request = self.cams[camera_name]["imaging"].create_type("Stop")
@@ -578,11 +555,6 @@ class OnvifController:
         self.cams[camera_name]["active"] = False
 
     async def _move_to_preset(self, camera_name: str, preset: str) -> None:
-        if isinstance(preset, bytes):
-            preset = preset.decode("utf-8")
-
-        preset = preset.lower()
-
         if preset not in self.cams[camera_name]["presets"]:
             logger.error(f"{preset} is not a valid preset for {camera_name}")
             return
@@ -676,7 +648,6 @@ class OnvifController:
         if (
             "focus" not in self.cams[camera_name]["features"]
             or not self.cams[camera_name]["video_source_token"]
-            or self.cams[camera_name]["imaging"] is None
         ):
             logger.error(f"{camera_name} does not support ONVIF continuous focus.")
             return

frigate/storage.py

@@ -5,7 +5,7 @@ import shutil
 import threading
 from pathlib import Path
 
-from peewee import SQL, fn
+from peewee import fn
 
 from frigate.config import FrigateConfig
 from frigate.const import RECORD_DIR
@@ -44,19 +44,13 @@ class StorageMaintainer(threading.Thread):
                     )
                 }
 
-                # calculate MB/hr from last 100 segments
+                # calculate MB/hr
                 try:
-                    # Subquery to get last 100 segments, then average their bandwidth
-                    last_100 = (
-                        Recordings.select(bandwidth_equation.alias("bw"))
-                        .where(Recordings.camera == camera, Recordings.segment_size > 0)
-                        .order_by(Recordings.start_time.desc())
-                        .limit(100)
-                        .alias("recent")
-                    )
-
                     bandwidth = round(
-                        Recordings.select(fn.AVG(SQL("bw"))).from_(last_100).scalar()
+                        Recordings.select(fn.AVG(bandwidth_equation))
+                        .where(Recordings.camera == camera, Recordings.segment_size > 0)
+                        .limit(100)
+                        .scalar()
                         * 3600,
                         2,
                     )

frigate/test/http_api/base_http_test.py

@@ -3,8 +3,6 @@ import logging
 import os
 import unittest
 
-from fastapi import Request
-from fastapi.testclient import TestClient
 from peewee_migrate import Router
 from playhouse.sqlite_ext import SqliteExtDatabase
 from playhouse.sqliteq import SqliteQueueDatabase
@@ -18,20 +16,6 @@ from frigate.review.types import SeverityEnum
 from frigate.test.const import TEST_DB, TEST_DB_CLEANUPS
 
 
-class AuthTestClient(TestClient):
-    """TestClient that automatically adds auth headers to all requests."""
-
-    def request(self, *args, **kwargs):
-        # Add default auth headers if not already present
-        headers = kwargs.get("headers") or {}
-        if "remote-user" not in headers:
-            headers["remote-user"] = "admin"
-        if "remote-role" not in headers:
-            headers["remote-role"] = "admin"
-        kwargs["headers"] = headers
-        return super().request(*args, **kwargs)
-
-
 class BaseTestHttp(unittest.TestCase):
     def setUp(self, models):
         # setup clean database for each test run
@@ -129,9 +113,7 @@ class BaseTestHttp(unittest.TestCase):
             pass
 
     def create_app(self, stats=None, event_metadata_publisher=None):
-        from frigate.api.auth import get_allowed_cameras_for_filter, get_current_user
-
-        app = create_fastapi_app(
+        return create_fastapi_app(
             FrigateConfig(**self.minimal_config),
             self.db,
             None,
@@ -141,33 +123,8 @@ class BaseTestHttp(unittest.TestCase):
             stats,
             event_metadata_publisher,
             None,
-            enforce_default_admin=False,
         )
 
-        # Default test mocks for authentication
-        # Tests can override these in their setUp if needed
-        # This mock uses headers set by AuthTestClient
-        async def mock_get_current_user(request: Request):
-            username = request.headers.get("remote-user")
-            role = request.headers.get("remote-role")
-            if not username or not role:
-                from fastapi.responses import JSONResponse
-
-                return JSONResponse(
-                    content={"message": "No authorization headers."}, status_code=401
-                )
-            return {"username": username, "role": role}
-
-        async def mock_get_allowed_cameras_for_filter(request: Request):
-            return list(self.minimal_config.get("cameras", {}).keys())
-
-        app.dependency_overrides[get_current_user] = mock_get_current_user
-        app.dependency_overrides[get_allowed_cameras_for_filter] = (
-            mock_get_allowed_cameras_for_filter
-        )
-
-        return app
-
     def insert_mock_event(
         self,
         id: str,

frigate/test/http_api/test_http_app.py

@@ -1,8 +1,10 @@
 from unittest.mock import Mock
 
+from fastapi.testclient import TestClient
+
 from frigate.models import Event, Recordings, ReviewSegment
 from frigate.stats.emitter import StatsEmitter
-from frigate.test.http_api.base_http_test import AuthTestClient, BaseTestHttp
+from frigate.test.http_api.base_http_test import BaseTestHttp
 
 
 class TestHttpApp(BaseTestHttp):
@@ -18,7 +20,7 @@ class TestHttpApp(BaseTestHttp):
         stats.get_latest_stats.return_value = self.test_stats
         app = super().create_app(stats)
 
-        with AuthTestClient(app) as client:
+        with TestClient(app) as client:
             response = client.get("/stats")
             response_json = response.json()
             assert response_json == self.test_stats

frigate/test/http_api/test_http_camera_access.py

@@ -1,13 +1,14 @@
 from unittest.mock import patch
 
 from fastapi import HTTPException, Request
+from fastapi.testclient import TestClient
 
 from frigate.api.auth import (
     get_allowed_cameras_for_filter,
     get_current_user,
 )
 from frigate.models import Event, Recordings, ReviewSegment
-from frigate.test.http_api.base_http_test import AuthTestClient, BaseTestHttp
+from frigate.test.http_api.base_http_test import BaseTestHttp
 
 
 class TestCameraAccessEventReview(BaseTestHttp):
@@ -15,17 +16,9 @@ class TestCameraAccessEventReview(BaseTestHttp):
         super().setUp([Event, ReviewSegment, Recordings])
         self.app = super().create_app()
 
-        # Mock get_current_user for all tests
-        async def mock_get_current_user(request: Request):
-            username = request.headers.get("remote-user")
-            role = request.headers.get("remote-role")
-            if not username or not role:
-                from fastapi.responses import JSONResponse
-
-                return JSONResponse(
-                    content={"message": "No authorization headers."}, status_code=401
-                )
-            return {"username": username, "role": role}
+        # Mock get_current_user to return valid user for all tests
+        async def mock_get_current_user():
+            return {"username": "test_user", "role": "user"}
 
         self.app.dependency_overrides[get_current_user] = mock_get_current_user
 
@@ -37,25 +30,21 @@ class TestCameraAccessEventReview(BaseTestHttp):
         super().insert_mock_event("event1", camera="front_door")
         super().insert_mock_event("event2", camera="back_door")
 
-        async def mock_cameras(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events")
             assert resp.status_code == 200
             ids = [e["id"] for e in resp.json()]
             assert "event1" in ids
             assert "event2" not in ids
 
-        async def mock_cameras(request: Request):
-            return [
-                "front_door",
-                "back_door",
-            ]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door",
+            "back_door",
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events")
             assert resp.status_code == 200
             ids = [e["id"] for e in resp.json()]
@@ -65,25 +54,21 @@ class TestCameraAccessEventReview(BaseTestHttp):
         super().insert_mock_review_segment("rev1", camera="front_door")
         super().insert_mock_review_segment("rev2", camera="back_door")
 
-        async def mock_cameras(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/review")
             assert resp.status_code == 200
             ids = [r["id"] for r in resp.json()]
             assert "rev1" in ids
             assert "rev2" not in ids
 
-        async def mock_cameras(request: Request):
-            return [
-                "front_door",
-                "back_door",
-            ]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door",
+            "back_door",
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/review")
             assert resp.status_code == 200
             ids = [r["id"] for r in resp.json()]
@@ -99,7 +84,7 @@ class TestCameraAccessEventReview(BaseTestHttp):
             raise HTTPException(status_code=403, detail="Access denied")
 
         with patch("frigate.api.event.require_camera_access", mock_require_allowed):
-            with AuthTestClient(self.app) as client:
+            with TestClient(self.app) as client:
                 resp = client.get("/events/event1")
                 assert resp.status_code == 200
                 assert resp.json()["id"] == "event1"
@@ -109,7 +94,7 @@ class TestCameraAccessEventReview(BaseTestHttp):
             raise HTTPException(status_code=403, detail="Access denied")
 
         with patch("frigate.api.event.require_camera_access", mock_require_disallowed):
-            with AuthTestClient(self.app) as client:
+            with TestClient(self.app) as client:
                 resp = client.get("/events/event1")
                 assert resp.status_code == 403
 
@@ -123,7 +108,7 @@ class TestCameraAccessEventReview(BaseTestHttp):
             raise HTTPException(status_code=403, detail="Access denied")
 
         with patch("frigate.api.review.require_camera_access", mock_require_allowed):
-            with AuthTestClient(self.app) as client:
+            with TestClient(self.app) as client:
                 resp = client.get("/review/rev1")
                 assert resp.status_code == 200
                 assert resp.json()["id"] == "rev1"
@@ -133,7 +118,7 @@ class TestCameraAccessEventReview(BaseTestHttp):
             raise HTTPException(status_code=403, detail="Access denied")
 
         with patch("frigate.api.review.require_camera_access", mock_require_disallowed):
-            with AuthTestClient(self.app) as client:
+            with TestClient(self.app) as client:
                 resp = client.get("/review/rev1")
                 assert resp.status_code == 403
 
@@ -141,25 +126,21 @@ class TestCameraAccessEventReview(BaseTestHttp):
         super().insert_mock_event("event1", camera="front_door")
         super().insert_mock_event("event2", camera="back_door")
 
-        async def mock_cameras(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events", params={"cameras": "all"})
             assert resp.status_code == 200
             ids = [e["id"] for e in resp.json()]
             assert "event1" in ids
             assert "event2" not in ids
 
-        async def mock_cameras(request: Request):
-            return [
-                "front_door",
-                "back_door",
-            ]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door",
+            "back_door",
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events", params={"cameras": "all"})
             assert resp.status_code == 200
             ids = [e["id"] for e in resp.json()]
@@ -169,24 +150,20 @@ class TestCameraAccessEventReview(BaseTestHttp):
         super().insert_mock_event("event1", camera="front_door")
         super().insert_mock_event("event2", camera="back_door")
 
-        async def mock_cameras(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events/summary")
             assert resp.status_code == 200
             summary_list = resp.json()
             assert len(summary_list) == 1
 
-        async def mock_cameras(request: Request):
-            return [
-                "front_door",
-                "back_door",
-            ]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = mock_cameras
-        with AuthTestClient(self.app) as client:
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door",
+            "back_door",
+        ]
+        with TestClient(self.app) as client:
             resp = client.get("/events/summary")
             summary_list = resp.json()
             assert len(summary_list) == 2

frigate/test/http_api/test_http_event.py

@@ -2,13 +2,14 @@ from datetime import datetime
 from typing import Any
 from unittest.mock import Mock
 
+from fastapi.testclient import TestClient
 from playhouse.shortcuts import model_to_dict
 
 from frigate.api.auth import get_allowed_cameras_for_filter, get_current_user
 from frigate.comms.event_metadata_updater import EventMetadataPublisher
 from frigate.models import Event, Recordings, ReviewSegment, Timeline
 from frigate.stats.emitter import StatsEmitter
-from frigate.test.http_api.base_http_test import AuthTestClient, BaseTestHttp, Request
+from frigate.test.http_api.base_http_test import BaseTestHttp
 from frigate.test.test_storage import _insert_mock_event
 
 
@@ -17,26 +18,14 @@ class TestHttpApp(BaseTestHttp):
         super().setUp([Event, Recordings, ReviewSegment, Timeline])
         self.app = super().create_app()
 
-        # Mock get_current_user for all tests
-        async def mock_get_current_user(request: Request):
-            username = request.headers.get("remote-user")
-            role = request.headers.get("remote-role")
-            if not username or not role:
-                from fastapi.responses import JSONResponse
-
-                return JSONResponse(
-                    content={"message": "No authorization headers."}, status_code=401
-                )
-            return {"username": username, "role": role}
+        # Mock auth to bypass camera access for tests
+        async def mock_get_current_user(request: Any):
+            return {"username": "test_user", "role": "admin"}
 
         self.app.dependency_overrides[get_current_user] = mock_get_current_user
-
-        async def mock_get_allowed_cameras_for_filter(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = (
-            mock_get_allowed_cameras_for_filter
-        )
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
 
     def tearDown(self):
         self.app.dependency_overrides.clear()
@@ -46,20 +35,20 @@ class TestHttpApp(BaseTestHttp):
     ###################################  GET /events Endpoint   #########################################################
     ####################################################################################################################
     def test_get_event_list_no_events(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             events = client.get("/events").json()
             assert len(events) == 0
 
     def test_get_event_list_no_match_event_id(self):
         id = "123456.random"
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             events = client.get("/events", params={"event_id": "abc"}).json()
             assert len(events) == 0
 
     def test_get_event_list_match_event_id(self):
         id = "123456.random"
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             events = client.get("/events", params={"event_id": id}).json()
             assert len(events) == 1
@@ -69,7 +58,7 @@ class TestHttpApp(BaseTestHttp):
         now = int(datetime.now().timestamp())
 
         id = "123456.random"
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id, now, now + 1)
             events = client.get(
                 "/events", params={"max_length": 1, "min_length": 1}
@@ -80,7 +69,7 @@ class TestHttpApp(BaseTestHttp):
     def test_get_event_list_no_match_max_length(self):
         now = int(datetime.now().timestamp())
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_event(id, now, now + 2)
             events = client.get("/events", params={"max_length": 1}).json()
@@ -89,7 +78,7 @@ class TestHttpApp(BaseTestHttp):
     def test_get_event_list_no_match_min_length(self):
         now = int(datetime.now().timestamp())
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_event(id, now, now + 2)
             events = client.get("/events", params={"min_length": 3}).json()
@@ -99,7 +88,7 @@ class TestHttpApp(BaseTestHttp):
         id = "123456.random"
         id2 = "54321.random"
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             events = client.get("/events").json()
             assert len(events) == 1
@@ -119,14 +108,14 @@ class TestHttpApp(BaseTestHttp):
     def test_get_event_list_no_match_has_clip(self):
         now = int(datetime.now().timestamp())
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_event(id, now, now + 2)
             events = client.get("/events", params={"has_clip": 0}).json()
             assert len(events) == 0
 
     def test_get_event_list_has_clip(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_event(id, has_clip=True)
             events = client.get("/events", params={"has_clip": 1}).json()
@@ -134,7 +123,7 @@ class TestHttpApp(BaseTestHttp):
             assert events[0]["id"] == id
 
     def test_get_event_list_sort_score(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             id2 = "54321.random"
             super().insert_mock_event(id, top_score=37, score=37, data={"score": 50})
@@ -152,7 +141,7 @@ class TestHttpApp(BaseTestHttp):
     def test_get_event_list_sort_start_time(self):
         now = int(datetime.now().timestamp())
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             id2 = "54321.random"
             super().insert_mock_event(id, start_time=now + 3)
@@ -170,7 +159,7 @@ class TestHttpApp(BaseTestHttp):
     def test_get_good_event(self):
         id = "123456.random"
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             event = client.get(f"/events/{id}").json()
 
@@ -182,7 +171,7 @@ class TestHttpApp(BaseTestHttp):
         id = "123456.random"
         bad_id = "654321.other"
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             event_response = client.get(f"/events/{bad_id}")
             assert event_response.status_code == 404
@@ -191,7 +180,7 @@ class TestHttpApp(BaseTestHttp):
     def test_delete_event(self):
         id = "123456.random"
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             event = client.get(f"/events/{id}").json()
             assert event
@@ -204,7 +193,7 @@ class TestHttpApp(BaseTestHttp):
     def test_event_retention(self):
         id = "123456.random"
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(id)
             client.post(f"/events/{id}/retain", headers={"remote-role": "admin"})
             event = client.get(f"/events/{id}").json()
@@ -223,11 +212,12 @@ class TestHttpApp(BaseTestHttp):
         morning = 1656590400  # 06/30/2022 6 am (GMT)
         evening = 1656633600  # 06/30/2022 6 pm (GMT)
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_event(morning_id, morning)
             super().insert_mock_event(evening_id, evening)
             # both events come back
             events = client.get("/events").json()
+            print("events!!!", events)
             assert events
             assert len(events) == 2
             # morning event is excluded
@@ -258,7 +248,7 @@ class TestHttpApp(BaseTestHttp):
 
         mock_event_updater.publish.side_effect = update_event
 
-        with AuthTestClient(app) as client:
+        with TestClient(app) as client:
             super().insert_mock_event(id)
             new_sub_label_response = client.post(
                 f"/events/{id}/sub_label",
@@ -295,7 +285,7 @@ class TestHttpApp(BaseTestHttp):
 
         mock_event_updater.publish.side_effect = update_event
 
-        with AuthTestClient(app) as client:
+        with TestClient(app) as client:
             super().insert_mock_event(id)
             client.post(
                 f"/events/{id}/sub_label",
@@ -311,7 +301,7 @@ class TestHttpApp(BaseTestHttp):
     ####################################################################################################################
     def test_get_metrics(self):
         """ensure correct prometheus metrics api response"""
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             ts_start = datetime.now().timestamp()
             ts_end = ts_start + 30
             _insert_mock_event(

frigate/test/http_api/test_http_media.py

@@ -1,13 +1,14 @@
 """Unit tests for recordings/media API endpoints."""
 
 from datetime import datetime, timezone
+from typing import Any
 
 import pytz
-from fastapi import Request
+from fastapi.testclient import TestClient
 
 from frigate.api.auth import get_allowed_cameras_for_filter, get_current_user
 from frigate.models import Recordings
-from frigate.test.http_api.base_http_test import AuthTestClient, BaseTestHttp
+from frigate.test.http_api.base_http_test import BaseTestHttp
 
 
 class TestHttpMedia(BaseTestHttp):
@@ -18,26 +19,15 @@ class TestHttpMedia(BaseTestHttp):
         super().setUp([Recordings])
         self.app = super().create_app()
 
-        # Mock get_current_user for all tests
-        async def mock_get_current_user(request: Request):
-            username = request.headers.get("remote-user")
-            role = request.headers.get("remote-role")
-            if not username or not role:
-                from fastapi.responses import JSONResponse
-
-                return JSONResponse(
-                    content={"message": "No authorization headers."}, status_code=401
-                )
-            return {"username": username, "role": role}
+        # Mock auth to bypass camera access for tests
+        async def mock_get_current_user(request: Any):
+            return {"username": "test_user", "role": "admin"}
 
         self.app.dependency_overrides[get_current_user] = mock_get_current_user
-
-        async def mock_get_allowed_cameras_for_filter(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = (
-            mock_get_allowed_cameras_for_filter
-        )
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door",
+            "back_door",
+        ]
 
     def tearDown(self):
         """Clean up after tests."""
@@ -62,7 +52,7 @@ class TestHttpMedia(BaseTestHttp):
         # March 11, 2024 at 12:00 PM EDT (after DST)
         march_11_noon = tz.localize(datetime(2024, 3, 11, 12, 0, 0)).timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             # Insert recordings for each day
             Recordings.insert(
                 id="recording_march_9",
@@ -138,7 +128,7 @@ class TestHttpMedia(BaseTestHttp):
         # November 4, 2024 at 12:00 PM EST (after DST)
         nov_4_noon = tz.localize(datetime(2024, 11, 4, 12, 0, 0)).timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             # Insert recordings for each day
             Recordings.insert(
                 id="recording_nov_2",
@@ -205,15 +195,7 @@ class TestHttpMedia(BaseTestHttp):
         # March 10, 2024 at 3:00 PM EDT (after DST transition)
         march_10_afternoon = tz.localize(datetime(2024, 3, 10, 15, 0, 0)).timestamp()
 
-        with AuthTestClient(self.app) as client:
-            # Override allowed cameras for this test to include both
-            async def mock_get_allowed_cameras_for_filter(_request: Request):
-                return ["front_door", "back_door"]
-
-            self.app.dependency_overrides[get_allowed_cameras_for_filter] = (
-                mock_get_allowed_cameras_for_filter
-            )
-
+        with TestClient(self.app) as client:
             # Insert recordings for front_door on March 9
             Recordings.insert(
                 id="front_march_9",
@@ -254,14 +236,6 @@ class TestHttpMedia(BaseTestHttp):
             assert summary["2024-03-09"] is True
             assert summary["2024-03-10"] is True
 
-            # Reset dependency override back to default single camera for other tests
-            async def reset_allowed_cameras(_request: Request):
-                return ["front_door"]
-
-            self.app.dependency_overrides[get_allowed_cameras_for_filter] = (
-                reset_allowed_cameras
-            )
-
     def test_recordings_summary_at_dst_transition_time(self):
         """
         Test recordings that span the exact DST transition time.
@@ -276,7 +250,7 @@ class TestHttpMedia(BaseTestHttp):
         # This is 1.5 hours of actual time but spans the "missing" hour
         after_transition = tz.localize(datetime(2024, 3, 10, 3, 30, 0)).timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             Recordings.insert(
                 id="recording_during_transition",
                 path="/media/recordings/transition.mp4",
@@ -309,7 +283,7 @@ class TestHttpMedia(BaseTestHttp):
         march_9_utc = datetime(2024, 3, 9, 17, 0, 0, tzinfo=timezone.utc).timestamp()
         march_10_utc = datetime(2024, 3, 10, 17, 0, 0, tzinfo=timezone.utc).timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             Recordings.insert(
                 id="recording_march_9_utc",
                 path="/media/recordings/march_9_utc.mp4",
@@ -351,7 +325,7 @@ class TestHttpMedia(BaseTestHttp):
         """
         Test recordings summary when no recordings exist.
         """
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             response = client.get(
                 "/recordings/summary",
                 params={"timezone": "America/New_York", "cameras": "all"},
@@ -368,7 +342,7 @@ class TestHttpMedia(BaseTestHttp):
         tz = pytz.timezone("America/New_York")
         march_10_noon = tz.localize(datetime(2024, 3, 10, 12, 0, 0)).timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             # Insert recordings for both cameras
             Recordings.insert(
                 id="front_recording",

frigate/test/http_api/test_http_review.py

@@ -1,12 +1,12 @@
 from datetime import datetime, timedelta
 
-from fastapi import Request
+from fastapi.testclient import TestClient
 from peewee import DoesNotExist
 
 from frigate.api.auth import get_allowed_cameras_for_filter, get_current_user
 from frigate.models import Event, Recordings, ReviewSegment, UserReviewStatus
 from frigate.review.types import SeverityEnum
-from frigate.test.http_api.base_http_test import AuthTestClient, BaseTestHttp
+from frigate.test.http_api.base_http_test import BaseTestHttp
 
 
 class TestHttpReview(BaseTestHttp):
@@ -16,26 +16,14 @@ class TestHttpReview(BaseTestHttp):
         self.user_id = "admin"
 
         # Mock get_current_user for all tests
-        # This mock uses headers set by AuthTestClient
-        async def mock_get_current_user(request: Request):
-            username = request.headers.get("remote-user")
-            role = request.headers.get("remote-role")
-            if not username or not role:
-                from fastapi.responses import JSONResponse
-
-                return JSONResponse(
-                    content={"message": "No authorization headers."}, status_code=401
-                )
-            return {"username": username, "role": role}
+        async def mock_get_current_user():
+            return {"username": self.user_id, "role": "admin"}
 
         self.app.dependency_overrides[get_current_user] = mock_get_current_user
 
-        async def mock_get_allowed_cameras_for_filter(request: Request):
-            return ["front_door"]
-
-        self.app.dependency_overrides[get_allowed_cameras_for_filter] = (
-            mock_get_allowed_cameras_for_filter
-        )
+        self.app.dependency_overrides[get_allowed_cameras_for_filter] = lambda: [
+            "front_door"
+        ]
 
     def tearDown(self):
         self.app.dependency_overrides.clear()
@@ -69,7 +57,7 @@ class TestHttpReview(BaseTestHttp):
         but ends after is included in the results."""
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random", now, now + 2)
             response = client.get("/review")
             assert response.status_code == 200
@@ -79,7 +67,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_no_filters(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now - 2, now - 1)
             response = client.get("/review")
@@ -93,7 +81,7 @@ class TestHttpReview(BaseTestHttp):
         """Test that review items outside the range are not returned."""
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now - 2, now - 1)
             super().insert_mock_review_segment(f"{id}2", now + 4, now + 5)
@@ -109,7 +97,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_with_time_filter(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now, now + 2)
             params = {
@@ -125,7 +113,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_with_limit_filter(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             id2 = "654321.random"
             super().insert_mock_review_segment(id, now, now + 2)
@@ -144,7 +132,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_with_severity_filters_no_matches(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now, now + 2, SeverityEnum.detection)
             params = {
@@ -161,7 +149,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_with_severity_filters(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now, now + 2, SeverityEnum.detection)
             params = {
@@ -177,7 +165,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_with_all_filters(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now, now + 2)
             params = {
@@ -200,7 +188,7 @@ class TestHttpReview(BaseTestHttp):
     ###################################  GET /review/summary Endpoint   #################################################
     ####################################################################################################################
     def test_get_review_summary_all_filters(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             params = {
                 "cameras": "front_door",
@@ -231,7 +219,7 @@ class TestHttpReview(BaseTestHttp):
             self.assertEqual(response_json, expected_response)
 
     def test_get_review_summary_no_filters(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             response = client.get("/review/summary")
             assert response.status_code == 200
@@ -259,7 +247,7 @@ class TestHttpReview(BaseTestHttp):
         now = datetime.now()
         five_days_ago = datetime.today() - timedelta(days=5)
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment(
                 "123456.random", now.timestamp() - 2, now.timestamp() - 1
             )
@@ -303,7 +291,7 @@ class TestHttpReview(BaseTestHttp):
         now = datetime.now()
         five_days_ago = datetime.today() - timedelta(days=5)
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random", now.timestamp())
             five_days_ago_ts = five_days_ago.timestamp()
             for i in range(20):
@@ -354,7 +342,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review_summary_multiple_in_same_day_with_reviewed(self):
         five_days_ago = datetime.today() - timedelta(days=5)
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             five_days_ago_ts = five_days_ago.timestamp()
             for i in range(10):
                 id = f"123456_{i}.random_alert_not_reviewed"
@@ -405,14 +393,14 @@ class TestHttpReview(BaseTestHttp):
     ####################################################################################################################
 
     def test_post_reviews_viewed_no_body(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             response = client.post("/reviews/viewed")
             # Missing ids
             assert response.status_code == 422
 
     def test_post_reviews_viewed_no_body_ids(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             body = {"ids": [""]}
             response = client.post("/reviews/viewed", json=body)
@@ -420,7 +408,7 @@ class TestHttpReview(BaseTestHttp):
             assert response.status_code == 422
 
     def test_post_reviews_viewed_non_existent_id(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id)
             body = {"ids": ["1"]}
@@ -437,7 +425,7 @@ class TestHttpReview(BaseTestHttp):
                 )
 
     def test_post_reviews_viewed(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id)
             body = {"ids": [id]}
@@ -457,14 +445,14 @@ class TestHttpReview(BaseTestHttp):
     ###################################  POST reviews/delete Endpoint   ################################################
     ####################################################################################################################
     def test_post_reviews_delete_no_body(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             response = client.post("/reviews/delete", headers={"remote-role": "admin"})
             # Missing ids
             assert response.status_code == 422
 
     def test_post_reviews_delete_no_body_ids(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             super().insert_mock_review_segment("123456.random")
             body = {"ids": [""]}
             response = client.post(
@@ -474,7 +462,7 @@ class TestHttpReview(BaseTestHttp):
             assert response.status_code == 422
 
     def test_post_reviews_delete_non_existent_id(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id)
             body = {"ids": ["1"]}
@@ -491,7 +479,7 @@ class TestHttpReview(BaseTestHttp):
             assert review_ids_in_db_after[0].id == id
 
     def test_post_reviews_delete(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id)
             body = {"ids": [id]}
@@ -507,7 +495,7 @@ class TestHttpReview(BaseTestHttp):
             assert len(review_ids_in_db_after) == 0
 
     def test_post_reviews_delete_many(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             ids = ["123456.random", "654321.random"]
             for id in ids:
                 super().insert_mock_review_segment(id)
@@ -539,7 +527,7 @@ class TestHttpReview(BaseTestHttp):
     def test_review_activity_motion_no_data_for_time_range(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             params = {
                 "after": now,
                 "before": now + 3,
@@ -552,7 +540,7 @@ class TestHttpReview(BaseTestHttp):
     def test_review_activity_motion(self):
         now = int(datetime.now().timestamp())
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             one_m = int((datetime.now() + timedelta(minutes=1)).timestamp())
             id = "123456.random"
             id2 = "123451.random"
@@ -585,7 +573,7 @@ class TestHttpReview(BaseTestHttp):
     ###################################  GET /review/event/{event_id} Endpoint   #######################################
     ####################################################################################################################
     def test_review_event_not_found(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             response = client.get("/review/event/123456.random")
             assert response.status_code == 404
             response_json = response.json()
@@ -597,7 +585,7 @@ class TestHttpReview(BaseTestHttp):
     def test_review_event_not_found_in_data(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             id = "123456.random"
             super().insert_mock_review_segment(id, now + 1, now + 2)
             response = client.get(f"/review/event/{id}")
@@ -611,7 +599,7 @@ class TestHttpReview(BaseTestHttp):
     def test_review_get_specific_event(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             event_id = "123456.event.random"
             super().insert_mock_event(event_id)
             review_id = "123456.review.random"
@@ -638,7 +626,7 @@ class TestHttpReview(BaseTestHttp):
     ###################################  GET /review/{review_id} Endpoint   #######################################
     ####################################################################################################################
     def test_review_not_found(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             response = client.get("/review/123456.random")
             assert response.status_code == 404
             response_json = response.json()
@@ -650,7 +638,7 @@ class TestHttpReview(BaseTestHttp):
     def test_get_review(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             review_id = "123456.review.random"
             super().insert_mock_review_segment(review_id, now + 1, now + 2)
             response = client.get(f"/review/{review_id}")
@@ -674,7 +662,7 @@ class TestHttpReview(BaseTestHttp):
     ####################################################################################################################
 
     def test_delete_review_viewed_review_not_found(self):
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             review_id = "123456.random"
             response = client.delete(f"/review/{review_id}/viewed")
             assert response.status_code == 404
@@ -687,7 +675,7 @@ class TestHttpReview(BaseTestHttp):
     def test_delete_review_viewed(self):
         now = datetime.now().timestamp()
 
-        with AuthTestClient(self.app) as client:
+        with TestClient(self.app) as client:
             review_id = "123456.review.random"
             super().insert_mock_review_segment(review_id, now + 1, now + 2)
             self._insert_user_review_status(review_id, reviewed=True)

frigate/track/object_processing.py

@@ -78,8 +78,6 @@ class TrackedObjectProcessor(threading.Thread):
             [
                 CameraConfigUpdateEnum.add,
                 CameraConfigUpdateEnum.enabled,
-                CameraConfigUpdateEnum.motion,
-                CameraConfigUpdateEnum.objects,
                 CameraConfigUpdateEnum.remove,
                 CameraConfigUpdateEnum.zones,
             ],

frigate/util/classification.py

@@ -330,7 +330,7 @@ def collect_state_classification_examples(
     1. Queries review items from specified cameras
     2. Selects 100 balanced timestamps across the data
     3. Extracts keyframes from recordings (cropped to specified regions)
-    4. Selects 24 most visually distinct images
+    4. Selects 20 most visually distinct images
     5. Saves them to the dataset directory
 
     Args:
@@ -660,6 +660,7 @@ def collect_object_classification_examples(
     Args:
         model_name: Name of the classification model
         label: Object label to collect (e.g., "person", "car")
+        cameras: List of camera names to collect examples from
     """
     dataset_dir = os.path.join(CLIPS_DIR, model_name, "dataset")
     temp_dir = os.path.join(dataset_dir, "temp")

frigate/util/config.py

@@ -348,7 +348,7 @@ def migrate_016_0(config: dict[str, dict[str, Any]]) -> dict[str, dict[str, Any]
 
 
 def migrate_017_0(config: dict[str, dict[str, Any]]) -> dict[str, dict[str, Any]]:
-    """Handle migrating frigate config to 0.17-0"""
+    """Handle migrating frigate config to 0.16-0"""
     new_config = config.copy()
 
     # migrate global to new recording configuration
@@ -380,7 +380,7 @@ def migrate_017_0(config: dict[str, dict[str, Any]]) -> dict[str, dict[str, Any]
 
     if global_genai:
         new_genai_config = {}
-        new_object_config = new_config.get("objects", {})
+        new_object_config = config.get("objects", {})
         new_object_config["genai"] = {}
 
         for key in global_genai.keys():
@@ -389,8 +389,7 @@ def migrate_017_0(config: dict[str, dict[str, Any]]) -> dict[str, dict[str, Any]
             else:
                 new_object_config["genai"][key] = global_genai[key]
 
-        new_config["genai"] = new_genai_config
-        new_config["objects"] = new_object_config
+        config["genai"] = new_genai_config
 
     for name, camera in config.get("cameras", {}).items():
         camera_config: dict[str, dict[str, Any]] = camera.copy()
@@ -416,9 +415,8 @@ def migrate_017_0(config: dict[str, dict[str, Any]]) -> dict[str, dict[str, Any]
         camera_genai = camera_config.get("genai", {})
 
         if camera_genai:
-            camera_object_config = camera_config.get("objects", {})
-            camera_object_config["genai"] = camera_genai
-            camera_config["objects"] = camera_object_config
+            new_object_config = config.get("objects", {})
+            new_object_config["genai"] = camera_genai
             del camera_config["genai"]
 
         new_config["cameras"][name] = camera_config

frigate/util/process.py

@@ -1,10 +1,7 @@
-import atexit
 import faulthandler
 import logging
 import multiprocessing as mp
 import os
-import pathlib
-import subprocess
 import threading
 from logging.handlers import QueueHandler
 from multiprocessing.synchronize import Event as MpEvent
@@ -14,7 +11,6 @@ from setproctitle import setproctitle
 
 import frigate.log
 from frigate.config.logger import LoggerConfig
-from frigate.const import CONFIG_DIR
 
 
 class BaseProcess(mp.Process):
@@ -52,7 +48,6 @@ class FrigateProcess(BaseProcess):
 
     def before_start(self) -> None:
         self.__log_queue = frigate.log.log_listener.queue
-        self.__memray_tracker = None
 
     def pre_run_setup(self, logConfig: LoggerConfig | None = None) -> None:
         os.nice(self.priority)
@@ -69,86 +64,3 @@ class FrigateProcess(BaseProcess):
             frigate.log.apply_log_levels(
                 logConfig.default.value.upper(), logConfig.logs
             )
-
-        self._setup_memray()
-
-    def _setup_memray(self) -> None:
-        """Setup memray profiling if enabled via environment variable."""
-        memray_modules = os.environ.get("FRIGATE_MEMRAY_MODULES", "")
-
-        if not memray_modules:
-            return
-
-        # Extract module name from process name (e.g., "frigate.capture:camera" -> "frigate.capture")
-        process_name = self.name
-        module_name = (
-            process_name.split(":")[0] if ":" in process_name else process_name
-        )
-
-        enabled_modules = [m.strip() for m in memray_modules.split(",")]
-
-        if module_name not in enabled_modules and process_name not in enabled_modules:
-            return
-
-        try:
-            import memray
-
-            reports_dir = pathlib.Path(CONFIG_DIR) / "memray_reports"
-            reports_dir.mkdir(parents=True, exist_ok=True)
-            safe_name = (
-                process_name.replace(":", "_").replace("/", "_").replace("\\", "_")
-            )
-
-            binary_file = reports_dir / f"{safe_name}.bin"
-
-            self.__memray_tracker = memray.Tracker(str(binary_file))
-            self.__memray_tracker.__enter__()
-
-            # Register cleanup handler to stop tracking and generate HTML report
-            # atexit runs on normal exits and most signal-based terminations (SIGTERM, SIGINT)
-            # For hard kills (SIGKILL) or segfaults, the binary file is preserved for manual generation
-            atexit.register(self._cleanup_memray, safe_name, binary_file)
-
-            self.logger.info(
-                f"Memray profiling enabled for module {module_name} (process: {self.name}). "
-                f"Binary file (updated continuously): {binary_file}. "
-                f"HTML report will be generated on exit: {reports_dir}/{safe_name}.html. "
-                f"If process crashes, manually generate with: memray flamegraph {binary_file}"
-            )
-        except Exception as e:
-            self.logger.error(f"Failed to setup memray profiling: {e}", exc_info=True)
-
-    def _cleanup_memray(self, safe_name: str, binary_file: pathlib.Path) -> None:
-        """Stop memray tracking and generate HTML report."""
-        if self.__memray_tracker is None:
-            return
-
-        try:
-            self.__memray_tracker.__exit__(None, None, None)
-            self.__memray_tracker = None
-
-            reports_dir = pathlib.Path(CONFIG_DIR) / "memray_reports"
-            html_file = reports_dir / f"{safe_name}.html"
-
-            result = subprocess.run(
-                ["memray", "flamegraph", "--output", str(html_file), str(binary_file)],
-                capture_output=True,
-                text=True,
-                timeout=10,
-            )
-
-            if result.returncode == 0:
-                self.logger.info(f"Memray report generated: {html_file}")
-            else:
-                self.logger.error(
-                    f"Failed to generate memray report: {result.stderr}. "
-                    f"Binary file preserved at {binary_file} for manual generation."
-                )
-
-            # Keep the binary file for manual report generation if needed
-            # Users can run: memray flamegraph {binary_file}
-
-        except subprocess.TimeoutExpired:
-            self.logger.error("Memray report generation timed out")
-        except Exception as e:
-            self.logger.error(f"Failed to cleanup memray profiling: {e}", exc_info=True)

frigate/video.py

@@ -124,50 +124,45 @@ def capture_frames(
         config_subscriber.check_for_updates()
         return config.enabled
 
-    try:
-        while not stop_event.is_set():
-            if not get_enabled_state():
-                logger.debug(f"Stopping capture thread for disabled {config.name}")
+    while not stop_event.is_set():
+        if not get_enabled_state():
+            logger.debug(f"Stopping capture thread for disabled {config.name}")
+            break
+
+        fps.value = frame_rate.eps()
+        skipped_fps.value = skipped_eps.eps()
+        current_frame.value = datetime.now().timestamp()
+        frame_name = f"{config.name}_frame{frame_index}"
+        frame_buffer = frame_manager.write(frame_name)
+        try:
+            frame_buffer[:] = ffmpeg_process.stdout.read(frame_size)
+        except Exception:
+            # shutdown has been initiated
+            if stop_event.is_set():
                 break
 
-            fps.value = frame_rate.eps()
-            skipped_fps.value = skipped_eps.eps()
-            current_frame.value = datetime.now().timestamp()
-            frame_name = f"{config.name}_frame{frame_index}"
-            frame_buffer = frame_manager.write(frame_name)
-            try:
-                frame_buffer[:] = ffmpeg_process.stdout.read(frame_size)
-            except Exception:
-                # shutdown has been initiated
-                if stop_event.is_set():
-                    break
+            logger.error(f"{config.name}: Unable to read frames from ffmpeg process.")
 
+            if ffmpeg_process.poll() is not None:
                 logger.error(
-                    f"{config.name}: Unable to read frames from ffmpeg process."
+                    f"{config.name}: ffmpeg process is not running. exiting capture thread..."
                 )
+                break
 
-                if ffmpeg_process.poll() is not None:
-                    logger.error(
-                        f"{config.name}: ffmpeg process is not running. exiting capture thread..."
-                    )
-                    break
+            continue
 
-                continue
+        frame_rate.update()
 
-            frame_rate.update()
+        # don't lock the queue to check, just try since it should rarely be full
+        try:
+            # add to the queue
+            frame_queue.put((frame_name, current_frame.value), False)
+            frame_manager.close(frame_name)
+        except queue.Full:
+            # if the queue is full, skip this frame
+            skipped_eps.update()
 
-            # don't lock the queue to check, just try since it should rarely be full
-            try:
-                # add to the queue
-                frame_queue.put((frame_name, current_frame.value), False)
-                frame_manager.close(frame_name)
-            except queue.Full:
-                # if the queue is full, skip this frame
-                skipped_eps.update()
-
-            frame_index = 0 if frame_index == shm_frame_count - 1 else frame_index + 1
-    finally:
-        config_subscriber.stop()
+        frame_index = 0 if frame_index == shm_frame_count - 1 else frame_index + 1
 
 
 class CameraWatchdog(threading.Thread):
@@ -239,16 +234,6 @@ class CameraWatchdog(threading.Thread):
                 else:
                     self.ffmpeg_detect_process.wait()
 
-        # Wait for old capture thread to fully exit before starting a new one
-        if self.capture_thread is not None and self.capture_thread.is_alive():
-            self.logger.info("Waiting for capture thread to exit...")
-            self.capture_thread.join(timeout=5)
-
-            if self.capture_thread.is_alive():
-                self.logger.warning(
-                    f"Capture thread for {self.config.name} did not exit in time"
-                )
-
         self.logger.error(
             "The following ffmpeg logs include the last 100 lines prior to exit."
         )
@@ -771,7 +756,6 @@ def process_frames(
             camera_enabled = camera_config.enabled
 
         if "motion" in updated_configs:
-            motion_detector.config = camera_config.motion
             motion_detector.update_mask()
 
         if (

migrations/001_create_events_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/002_add_clip_snapshot.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/003_create_recordings_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/004_add_bbox_region_area.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/005_make_end_time_nullable.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/006_add_motion_active_objects.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/007_add_retain_indefinitely.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/008_add_sub_label.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/009_add_object_filter_ratio.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/010_add_plus_image_id.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/011_update_indexes.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/012_add_segment_size.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/013_create_timeline_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/014_event_updates_for_fp.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/015_event_refactor.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/016_sublabel_increase.py

@@ -4,8 +4,8 @@ from frigate.models import Event
 
 
 def migrate(migrator, database, fake=False, **kwargs):
-    migrator.change_fields(Event, sub_label=pw.CharField(max_length=100, null=True))
+    migrator.change_columns(Event, sub_label=pw.CharField(max_length=100, null=True))
 
 
 def rollback(migrator, database, fake=False, **kwargs):
-    migrator.change_fields(Event, sub_label=pw.CharField(max_length=20, null=True))
+    migrator.change_columns(Event, sub_label=pw.CharField(max_length=20, null=True))

migrations/017_update_indexes.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/018_add_dbfs.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/019_create_regions_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/020_update_index_recordings.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/021_create_previews_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/022_create_review_segment_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/023_add_regions.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/024_create_export_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/025_create_user_table.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/026_add_notification_tokens.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/027_create_explore_index.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/028_optional_event_thumbnail.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/029_add_user_role.py

@@ -5,7 +5,7 @@ Some examples (model - class or model name)::
     > Model = migrator.orm['model_name']            # Return model in current state by name
 
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model

migrations/030_create_user_review_status.py

@@ -8,7 +8,7 @@ Some examples (model - class or model_name)::
 
     > Model = migrator.orm['model_name']            # Return model in current state by name
     > migrator.sql(sql)                             # Run custom SQL
-    > migrator.run(func, *args, **kwargs)           # Run python code
+    > migrator.python(func, *args, **kwargs)        # Run python code
     > migrator.create_model(Model)                  # Create a model (could be used as decorator)
     > migrator.remove_model(model, cascade=True)    # Remove a model
     > migrator.add_fields(model, **fields)          # Add fields to a model
@@ -54,9 +54,7 @@ def migrate(migrator, database, fake=False, **kwargs):
 
     # Migrate existing has_been_reviewed data to UserReviewStatus for all users
     def migrate_data():
-        # Use raw SQL to avoid ORM issues with columns that don't exist yet
-        cursor = database.execute_sql('SELECT "username" FROM "user"')
-        all_users = cursor.fetchall()
+        all_users = list(User.select())
         if not all_users:
             return
 
@@ -65,7 +63,7 @@ def migrate(migrator, database, fake=False, **kwargs):
         )
         reviewed_segment_ids = [row[0] for row in cursor.fetchall()]
         # also migrate for anonymous (unauthenticated users)
-        usernames = [user[0] for user in all_users] + ["anonymous"]
+        usernames = [user.username for user in all_users] + ["anonymous"]
 
         for segment_id in reviewed_segment_ids:
             for username in usernames:
@@ -76,7 +74,7 @@ def migrate(migrator, database, fake=False, **kwargs):
                 )
 
     if not fake:  # Only run data migration if not faking
-        migrator.run(migrate_data)
+        migrator.python(migrate_data)
 
     migrator.sql('ALTER TABLE "reviewsegment" DROP COLUMN "has_been_reviewed"')