Bump protobuf from 3.20.3 to 5.29.3 in /docker/tensorrt

Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 3.20.3 to 5.29.3. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/protobuf_release.bzl) - [Commits](https://github.com/protocolbuffers/protobuf/compare/v3.20.3...v5.29.3) --- updated-dependencies: - dependency-name: protobuf dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
2025-01-10 11:13:18 +00:00
20 changed files with 86 additions and 100 deletions
--- a/benchmark.py
+++ b/benchmark.py
@@ -61,7 +61,7 @@ def start(id, num_detections, detection_queue, event):
    object_detector.cleanup()
    print(f"{id} - Processed for {duration:.2f} seconds.")
    print(f"{id} - FPS: {object_detector.fps.eps():.2f}")
-    print(f"{id} - Average frame processing time: {mean(frame_times) * 1000:.2f}ms")
+    print(f"{id} - Average frame processing time: {mean(frame_times)*1000:.2f}ms")


 ######
--- a/docker/rockchip/Dockerfile
+++ b/docker/rockchip/Dockerfile
@@ -22,6 +22,6 @@ ADD https://github.com/MarcA711/rknn-toolkit2/releases/download/v2.0.0/librknnrt

 RUN rm -rf /usr/lib/btbn-ffmpeg/bin/ffmpeg
 RUN rm -rf /usr/lib/btbn-ffmpeg/bin/ffprobe
-ADD --chmod=111 https://github.com/MarcA711/Rockchip-FFmpeg-Builds/releases/download/6.1-7/ffmpeg /usr/lib/ffmpeg/6.0/bin/
-ADD --chmod=111 https://github.com/MarcA711/Rockchip-FFmpeg-Builds/releases/download/6.1-7/ffprobe /usr/lib/ffmpeg/6.0/bin/
+ADD --chmod=111 https://github.com/MarcA711/Rockchip-FFmpeg-Builds/releases/download/6.1-5/ffmpeg /usr/lib/ffmpeg/6.0/bin/
+ADD --chmod=111 https://github.com/MarcA711/Rockchip-FFmpeg-Builds/releases/download/6.1-5/ffprobe /usr/lib/ffmpeg/6.0/bin/
 ENV PATH="/usr/lib/ffmpeg/6.0/bin/:${PATH}"
--- a/docker/tensorrt/requirements-amd64.txt
+++ b/docker/tensorrt/requirements-amd64.txt
@@ -11,4 +11,4 @@ nvidia-cudnn-cu11 == 8.6.0.*; platform_machine == 'x86_64'
 nvidia-cufft-cu11==10.*; platform_machine == 'x86_64'
 onnx==1.16.*; platform_machine == 'x86_64'
 onnxruntime-gpu==1.18.*; platform_machine == 'x86_64'
-protobuf==3.20.3; platform_machine == 'x86_64'
+protobuf==5.29.3; platform_machine == 'x86_64'
--- a/docker/tensorrt/requirements-models-arm64.txt
+++ b/docker/tensorrt/requirements-models-arm64.txt
@@ -1,3 +1,3 @@
 onnx == 1.14.0; platform_machine == 'aarch64'
-protobuf == 3.20.3; platform_machine == 'aarch64'
+protobuf == 5.29.3; platform_machine == 'aarch64'
 numpy == 1.23.*; platform_machine == 'aarch64'  # required by python-tensorrt 8.2.1 (Jetpack 4.6)
--- a/docs/docs/configuration/reference.md
+++ b/docs/docs/configuration/reference.md
@@ -548,8 +548,6 @@ genai:

 # Optional: Restream configuration
 # Uses https://github.com/AlexxIT/go2rtc (v1.9.2)
-# NOTE: The default go2rtc API port (1984) must be used,
-#       changing this port for the integrated go2rtc instance is not supported. 
 go2rtc:

 # Optional: Live stream configuration for WebUI.
--- a/docs/docs/integrations/home-assistant.md
+++ b/docs/docs/integrations/home-assistant.md
@@ -47,7 +47,7 @@ that card.

 ## Configuration

-When configuring the integration, you will be asked for the `URL` of your Frigate instance which can be pointed at the internal unauthenticated port (`5000`) or the authenticated port (`8971`) for your instance. This may look like `http://<host>:5000/`.
+When configuring the integration, you will be asked for the `URL` of your Frigate instance which needs to be pointed at the internal unauthenticated port (`5000`) for your instance. This may look like `http://<host>:5000/`.

 ### Docker Compose Examples

@@ -55,7 +55,7 @@ If you are running Home Assistant Core and Frigate with Docker Compose on the sa

 #### Home Assistant running with host networking

-It is not recommended to run Frigate in host networking mode. In this example, you would use `http://172.17.0.1:5000` or `http://172.17.0.1:8971` when configuring the integration.
+It is not recommended to run Frigate in host networking mode. In this example, you would use `http://172.17.0.1:5000` when configuring the integration.

 ```yaml
 services:
@@ -75,7 +75,7 @@ services:

 #### Home Assistant _not_ running with host networking or in a separate compose file

-In this example, it is recommended to connect to the authenticated port, for example, `http://frigate:8971` when configuring the integration. There is no need to map the port for the Frigate container.
+In this example, you would use `http://frigate:5000` when configuring the integration. There is no need to map the port for the Frigate container.

 ```yaml
 services:
@@ -103,15 +103,14 @@ If you are using HassOS with the addon, the URL should be one of the following d
 | Frigate NVR (Full Access)      | `http://ccab4aaf-frigate-fa:5000`      |
 | Frigate NVR Beta               | `http://ccab4aaf-frigate-beta:5000`    |
 | Frigate NVR Beta (Full Access) | `http://ccab4aaf-frigate-fa-beta:5000` |
-| Frigate NVR HailoRT Beta       | `http://ccab4aaf-frigate-hailo-beta:5000`    |

 ### Frigate running on a separate machine

-If you run Frigate on a separate device within your local network, Home Assistant will need access to port 8971.
+If you run Frigate on a separate device within your local network, Home Assistant will need access to port 5000.

 #### Local network

-Use `http://<frigate_device_ip>:8971` as the URL for the integration so that authentication is required.
+Use `http://<frigate_device_ip>:5000` as the URL for the integration. If you want to protect access to port 5000, you can use firewall rules to limit access to the device running Home Assistant.

 ```yaml
 services:
@@ -119,7 +118,7 @@ services:
    image: ghcr.io/blakeblackshear/frigate:stable
    ...
    ports:
-      - "8971:8971"
+      - "5000:5000"
      ...
 ```

@@ -196,30 +195,12 @@ To load a snapshot for a tracked object:
 https://HA_URL/api/frigate/notifications/<event-id>/snapshot.jpg
 ```

-To load a video clip of a tracked object using an Android device:
+To load a video clip of a tracked object:

 ```
 https://HA_URL/api/frigate/notifications/<event-id>/clip.mp4
 ```

-To load a video clip of a tracked object using an iOS device:
-
-```
-https://HA_URL/api/frigate/notifications/<event-id>/master.m3u8
-```
-
-To load a preview gif of a tracked object:
-
-```
-https://HA_URL/api/frigate/notifications/<event-id>/event_preview.gif
-```
-
-To load a preview gif of a review item:
-
-```
-https://HA_URL/api/frigate/notifications/<review-id>/review_preview.gif
-```
-
 <a name="streams"></a>

 ## RTSP stream
--- a/frigate/api/media.py
+++ b/frigate/api/media.py
@@ -133,15 +133,6 @@ def latest_frame(
        "regions": params.regions,
    }
    quality = params.quality
-    mime_type = extension
-
-    if extension == "png":
-        quality_params = None
-    elif extension == "webp":
-        quality_params = [int(cv2.IMWRITE_WEBP_QUALITY), quality]
-    else:
-        quality_params = [int(cv2.IMWRITE_JPEG_QUALITY), quality]
-        mime_type = "jpeg"

    if camera_name in request.app.frigate_config.cameras:
        frame = frame_processor.get_current_frame(camera_name, draw_options)
@@ -182,11 +173,13 @@ def latest_frame(

        frame = cv2.resize(frame, dsize=(width, height), interpolation=cv2.INTER_AREA)

-        ret, img = cv2.imencode(f".{extension}", frame, quality_params)
+        ret, img = cv2.imencode(
+            f".{extension}", frame, [int(cv2.IMWRITE_WEBP_QUALITY), quality]
+        )
        return Response(
            content=img.tobytes(),
-            media_type=f"image/{mime_type}",
-            headers={"Content-Type": f"image/{mime_type}", "Cache-Control": "no-store"},
+            media_type=f"image/{extension}",
+            headers={"Content-Type": f"image/{extension}", "Cache-Control": "no-store"},
        )
    elif camera_name == "birdseye" and request.app.frigate_config.birdseye.restream:
        frame = cv2.cvtColor(
@@ -199,11 +192,13 @@ def latest_frame(

        frame = cv2.resize(frame, dsize=(width, height), interpolation=cv2.INTER_AREA)

-        ret, img = cv2.imencode(f".{extension}", frame, quality_params)
+        ret, img = cv2.imencode(
+            f".{extension}", frame, [int(cv2.IMWRITE_WEBP_QUALITY), quality]
+        )
        return Response(
            content=img.tobytes(),
-            media_type=f"image/{mime_type}",
-            headers={"Content-Type": f"image/{mime_type}", "Cache-Control": "no-store"},
+            media_type=f"image/{extension}",
+            headers={"Content-Type": f"image/{extension}", "Cache-Control": "no-store"},
        )
    else:
        return JSONResponse(
@@ -246,7 +241,6 @@ def get_snapshot_from_recording(
        recording: Recordings = recording_query.get()
        time_in_segment = frame_time - recording.start_time
        codec = "png" if format == "png" else "mjpeg"
-        mime_type = "png" if format == "png" else "jpeg"
        config: FrigateConfig = request.app.frigate_config

        image_data = get_image_from_recording(
@@ -263,7 +257,7 @@ def get_snapshot_from_recording(
                ),
                status_code=404,
            )
-        return Response(image_data, headers={"Content-Type": f"image/{mime_type}"})
+        return Response(image_data, headers={"Content-Type": f"image/{format}"})
    except DoesNotExist:
        return JSONResponse(
            content={
--- a/frigate/comms/webpush.py
+++ b/frigate/comms/webpush.py
@@ -151,7 +151,7 @@ class WebPushClient(Communicator):  # type: ignore[misc]
        camera: str = payload["after"]["camera"]
        title = f"{', '.join(sorted_objects).replace('_', ' ').title()}{' was' if state == 'end' else ''} detected in {', '.join(payload['after']['data']['zones']).replace('_', ' ').title()}"
        message = f"Detected on {camera.replace('_', ' ').title()}"
-        image = f"{payload['after']['thumb_path'].replace('/media/frigate', '')}"
+        image = f'{payload["after"]["thumb_path"].replace("/media/frigate", "")}'

        # if event is ongoing open to live view otherwise open to recordings view
        direct_url = f"/review?id={reviewId}" if state == "end" else f"/#{camera}"
--- a/frigate/config/camera/zone.py
+++ b/frigate/config/camera/zone.py
@@ -85,7 +85,7 @@ class ZoneConfig(BaseModel):
            if explicit:
                self.coordinates = ",".join(
                    [
-                        f"{round(int(p.split(',')[0]) / frame_shape[1], 3)},{round(int(p.split(',')[1]) / frame_shape[0], 3)}"
+                        f'{round(int(p.split(",")[0]) / frame_shape[1], 3)},{round(int(p.split(",")[1]) / frame_shape[0], 3)}'
                        for p in coordinates
                    ]
                )
--- a/frigate/detectors/plugins/tensorrt.py
+++ b/frigate/detectors/plugins/tensorrt.py
@@ -219,19 +219,19 @@ class TensorRtDetector(DetectionApi):
        ]

    def __init__(self, detector_config: TensorRTDetectorConfig):
-        assert TRT_SUPPORT, (
-            f"TensorRT libraries not found, {DETECTOR_KEY} detector not present"
-        )
+        assert (
+            TRT_SUPPORT
+        ), f"TensorRT libraries not found, {DETECTOR_KEY} detector not present"

        (cuda_err,) = cuda.cuInit(0)
-        assert cuda_err == cuda.CUresult.CUDA_SUCCESS, (
-            f"Failed to initialize cuda {cuda_err}"
-        )
+        assert (
+            cuda_err == cuda.CUresult.CUDA_SUCCESS
+        ), f"Failed to initialize cuda {cuda_err}"
        err, dev_count = cuda.cuDeviceGetCount()
        logger.debug(f"Num Available Devices: {dev_count}")
-        assert detector_config.device < dev_count, (
-            f"Invalid TensorRT Device Config. Device {detector_config.device} Invalid."
-        )
+        assert (
+            detector_config.device < dev_count
+        ), f"Invalid TensorRT Device Config. Device {detector_config.device} Invalid."
        err, self.cu_ctx = cuda.cuCtxCreate(
            cuda.CUctx_flags.CU_CTX_MAP_HOST, detector_config.device
        )
--- a/frigate/ffmpeg_presets.py
+++ b/frigate/ffmpeg_presets.py
@@ -50,9 +50,16 @@ class LibvaGpuSelector:
        return ""


-LIBAV_VERSION = int(os.getenv("LIBAVFORMAT_VERSION_MAJOR", "59") or "59")
-FPS_VFR_PARAM = "-fps_mode vfr" if LIBAV_VERSION >= 59 else "-vsync 2"
-TIMEOUT_PARAM = "-timeout" if LIBAV_VERSION >= 59 else "-stimeout"
+FPS_VFR_PARAM = (
+    "-fps_mode vfr"
+    if int(os.getenv("LIBAVFORMAT_VERSION_MAJOR", "59") or "59") >= 59
+    else "-vsync 2"
+)
+TIMEOUT_PARAM = (
+    "-timeout"
+    if int(os.getenv("LIBAVFORMAT_VERSION_MAJOR", "59") or "59") >= 59
+    else "-stimeout"
+)

 _gpu_selector = LibvaGpuSelector()
 _user_agent_args = [
@@ -64,8 +71,8 @@ PRESETS_HW_ACCEL_DECODE = {
    "preset-rpi-64-h264": "-c:v:1 h264_v4l2m2m",
    "preset-rpi-64-h265": "-c:v:1 hevc_v4l2m2m",
    FFMPEG_HWACCEL_VAAPI: f"-hwaccel_flags allow_profile_mismatch -hwaccel vaapi -hwaccel_device {_gpu_selector.get_selected_gpu()} -hwaccel_output_format vaapi",
-    "preset-intel-qsv-h264": f"-hwaccel qsv -qsv_device {_gpu_selector.get_selected_gpu()} -hwaccel_output_format qsv -c:v h264_qsv{' -bsf:v dump_extra' if LIBAV_VERSION >= 61 else ''}",  # https://trac.ffmpeg.org/ticket/9766#comment:17
-    "preset-intel-qsv-h265": f"-load_plugin hevc_hw -hwaccel qsv -qsv_device {_gpu_selector.get_selected_gpu()} -hwaccel_output_format qsv{' -bsf:v dump_extra' if LIBAV_VERSION >= 61 else ''}",  # https://trac.ffmpeg.org/ticket/9766#comment:17
+    "preset-intel-qsv-h264": f"-hwaccel qsv -qsv_device {_gpu_selector.get_selected_gpu()} -hwaccel_output_format qsv -c:v h264_qsv -bsf:v dump_extra",  # https://trac.ffmpeg.org/ticket/9766#comment:17
+    "preset-intel-qsv-h265": f"-load_plugin hevc_hw -hwaccel qsv -qsv_device {_gpu_selector.get_selected_gpu()} -hwaccel_output_format qsv -c:v hevc_qsv -bsf:v dump_extra",  # https://trac.ffmpeg.org/ticket/9766#comment:17
    FFMPEG_HWACCEL_NVIDIA: "-hwaccel cuda -hwaccel_output_format cuda",
    "preset-jetson-h264": "-c:v h264_nvmpi -resize {1}x{2}",
    "preset-jetson-h265": "-c:v hevc_nvmpi -resize {1}x{2}",
--- a/frigate/plus.py
+++ b/frigate/plus.py
@@ -68,13 +68,11 @@ class PlusApi:
            or self._token_data["expires"] - datetime.datetime.now().timestamp() < 60
        ):
            if self.key is None:
-                raise Exception(
-                    "Plus API key not set. See https://docs.frigate.video/integrations/plus#set-your-api-key"
-                )
+                raise Exception("Plus API not activated")
            parts = self.key.split(":")
            r = requests.get(f"{self.host}/v1/auth/token", auth=(parts[0], parts[1]))
            if not r.ok:
-                raise Exception(f"Unable to refresh API token: {r.text}")
+                raise Exception("Unable to refresh API token")
            self._token_data = r.json()

    def _get_authorization_header(self) -> dict:
@@ -118,6 +116,15 @@ class PlusApi:
            logger.error(f"Failed to upload original: {r.status_code} {r.text}")
            raise Exception(r.text)

+        # resize and submit annotate
+        files = {"file": get_jpg_bytes(image, 640, 70)}
+        data = presigned_urls["annotate"]["fields"]
+        data["content-type"] = "image/jpeg"
+        r = requests.post(presigned_urls["annotate"]["url"], files=files, data=data)
+        if not r.ok:
+            logger.error(f"Failed to upload annotate: {r.status_code} {r.text}")
+            raise Exception(r.text)
+
        # resize and submit thumbnail
        files = {"file": get_jpg_bytes(image, 200, 70)}
        data = presigned_urls["thumbnail"]["fields"]
--- a/frigate/ptz/autotrack.py
+++ b/frigate/ptz/autotrack.py
@@ -135,7 +135,7 @@ class PtzMotionEstimator:

            try:
                logger.debug(
-                    f"{camera}: Motion estimator transformation: {self.coord_transformations.rel_to_abs([[0, 0]])}"
+                    f"{camera}: Motion estimator transformation: {self.coord_transformations.rel_to_abs([[0,0]])}"
                )
            except Exception:
                pass
@@ -471,7 +471,7 @@ class PtzAutoTracker:
                self.onvif.get_camera_status(camera)

            logger.info(
-                f"Calibration for {camera} in progress: {round((step / num_steps) * 100)}% complete"
+                f"Calibration for {camera} in progress: {round((step/num_steps)*100)}% complete"
            )

        self.calibrating[camera] = False
@@ -690,7 +690,7 @@ class PtzAutoTracker:
                            f"{camera}: Predicted movement time: {self._predict_movement_time(camera, pan, tilt)}"
                        )
                        logger.debug(
-                            f"{camera}: Actual movement time: {self.ptz_metrics[camera].stop_time.value - self.ptz_metrics[camera].start_time.value}"
+                            f"{camera}: Actual movement time: {self.ptz_metrics[camera].stop_time.value-self.ptz_metrics[camera].start_time.value}"
                        )

                    # save metrics for better estimate calculations
@@ -983,10 +983,10 @@ class PtzAutoTracker:
            logger.debug(f"{camera}: Zoom test: at max zoom: {at_max_zoom}")
            logger.debug(f"{camera}: Zoom test: at min zoom: {at_min_zoom}")
            logger.debug(
-                f"{camera}: Zoom test: zoom in hysteresis limit: {zoom_in_hysteresis} value: {AUTOTRACKING_ZOOM_IN_HYSTERESIS} original: {self.tracked_object_metrics[camera]['original_target_box']} max: {self.tracked_object_metrics[camera]['max_target_box']} target: {calculated_target_box if calculated_target_box else self.tracked_object_metrics[camera]['target_box']}"
+                f'{camera}: Zoom test: zoom in hysteresis limit: {zoom_in_hysteresis} value: {AUTOTRACKING_ZOOM_IN_HYSTERESIS} original: {self.tracked_object_metrics[camera]["original_target_box"]} max: {self.tracked_object_metrics[camera]["max_target_box"]} target: {calculated_target_box if calculated_target_box else self.tracked_object_metrics[camera]["target_box"]}'
            )
            logger.debug(
-                f"{camera}: Zoom test: zoom out hysteresis limit: {zoom_out_hysteresis} value: {AUTOTRACKING_ZOOM_OUT_HYSTERESIS} original: {self.tracked_object_metrics[camera]['original_target_box']} max: {self.tracked_object_metrics[camera]['max_target_box']} target: {calculated_target_box if calculated_target_box else self.tracked_object_metrics[camera]['target_box']}"
+                f'{camera}: Zoom test: zoom out hysteresis limit: {zoom_out_hysteresis} value: {AUTOTRACKING_ZOOM_OUT_HYSTERESIS} original: {self.tracked_object_metrics[camera]["original_target_box"]} max: {self.tracked_object_metrics[camera]["max_target_box"]} target: {calculated_target_box if calculated_target_box else self.tracked_object_metrics[camera]["target_box"]}'
            )

        # Zoom in conditions (and)
@@ -1069,7 +1069,7 @@ class PtzAutoTracker:
                pan = ((centroid_x / camera_width) - 0.5) * 2
                tilt = (0.5 - (centroid_y / camera_height)) * 2

-            logger.debug(f"{camera}: Original box: {obj.obj_data['box']}")
+            logger.debug(f'{camera}: Original box: {obj.obj_data["box"]}')
            logger.debug(f"{camera}: Predicted box: {tuple(predicted_box)}")
            logger.debug(
                f"{camera}: Velocity: {tuple(np.round(average_velocity).flatten().astype(int))}"
@@ -1179,7 +1179,7 @@ class PtzAutoTracker:
                    )
                    zoom = (ratio - 1) / (ratio + 1)
                    logger.debug(
-                        f"{camera}: limit: {self.tracked_object_metrics[camera]['max_target_box']}, ratio: {ratio} zoom calculation: {zoom}"
+                        f'{camera}: limit: {self.tracked_object_metrics[camera]["max_target_box"]}, ratio: {ratio} zoom calculation: {zoom}'
                    )
                    if not result:
                        # zoom out with special condition if zooming out because of velocity, edges, etc.
--- a/frigate/record/maintainer.py
+++ b/frigate/record/maintainer.py
@@ -449,7 +449,7 @@ class RecordingMaintainer(threading.Thread):
                    return None
                else:
                    logger.debug(
-                        f"Copied {file_path} in {datetime.datetime.now().timestamp() - start_frame} seconds."
+                        f"Copied {file_path} in {datetime.datetime.now().timestamp()-start_frame} seconds."
                    )

                try:
--- a/frigate/review/maintainer.py
+++ b/frigate/review/maintainer.py
@@ -256,7 +256,7 @@ class ReviewSegmentMaintainer(threading.Thread):
                elif object["sub_label"][0] in self.config.model.all_attributes:
                    segment.detections[object["id"]] = object["sub_label"][0]
                else:
-                    segment.detections[object["id"]] = f"{object['label']}-verified"
+                    segment.detections[object["id"]] = f'{object["label"]}-verified'
                    segment.sub_labels[object["id"]] = object["sub_label"][0]

                # if object is alert label
@@ -352,7 +352,7 @@ class ReviewSegmentMaintainer(threading.Thread):
                elif object["sub_label"][0] in self.config.model.all_attributes:
                    detections[object["id"]] = object["sub_label"][0]
                else:
-                    detections[object["id"]] = f"{object['label']}-verified"
+                    detections[object["id"]] = f'{object["label"]}-verified'
                    sub_labels[object["id"]] = object["sub_label"][0]

                # if object is alert label
@@ -527,9 +527,7 @@ class ReviewSegmentMaintainer(threading.Thread):

                        if event_id in self.indefinite_events[camera]:
                            self.indefinite_events[camera].pop(event_id)
-
-                            if len(self.indefinite_events[camera]) == 0:
-                                current_segment.last_update = manual_info["end_time"]
+                            current_segment.last_update = manual_info["end_time"]
                        else:
                            logger.error(
                                f"Event with ID {event_id} has a set duration and can not be ended manually."
--- a/frigate/service_manager/multiprocessing.py
+++ b/frigate/service_manager/multiprocessing.py
@@ -72,7 +72,8 @@ class BaseServiceProcess(Service, ABC):
                running = False
            except TimeoutError:
                self.manager.logger.warning(
-                    f"{self.name} is still running after {timeout} seconds. Killing."
+                    f"{self.name} is still running after "
+                    f"{timeout} seconds. Killing."
                )

        if running:
--- a/frigate/track/tracked_object.py
+++ b/frigate/track/tracked_object.py
@@ -339,7 +339,7 @@ class TrackedObject:
                box[2],
                box[3],
                self.obj_data["label"],
-                f"{int(self.thumbnail_data['score'] * 100)}% {int(self.thumbnail_data['area'])}",
+                f"{int(self.thumbnail_data['score']*100)}% {int(self.thumbnail_data['area'])}",
                thickness=thickness,
                color=color,
            )
--- a/frigate/util/config.py
+++ b/frigate/util/config.py
@@ -314,7 +314,7 @@ def get_relative_coordinates(
                            continue

                        rel_points.append(
-                            f"{round(x / frame_shape[1], 3)},{round(y / frame_shape[0], 3)}"
+                            f"{round(x / frame_shape[1], 3)},{round(y  / frame_shape[0], 3)}"
                        )

                    relative_masks.append(",".join(rel_points))
@@ -337,7 +337,7 @@ def get_relative_coordinates(
                    return []

                rel_points.append(
-                    f"{round(x / frame_shape[1], 3)},{round(y / frame_shape[0], 3)}"
+                    f"{round(x / frame_shape[1], 3)},{round(y  / frame_shape[0], 3)}"
                )

            mask = ",".join(rel_points)
--- a/process_clip.py
+++ b/process_clip.py
@@ -208,7 +208,7 @@ class ProcessClip:
                box[2],
                box[3],
                obj["id"],
-                f"{int(obj['score'] * 100)}% {int(obj['area'])}",
+                f"{int(obj['score']*100)}% {int(obj['area'])}",
                thickness=thickness,
                color=color,
            )
@@ -227,7 +227,7 @@ class ProcessClip:
            )

        cv2.imwrite(
-            f"{os.path.join(debug_path, os.path.basename(self.clip_path))}.{int(frame_time * 1000000)}.jpg",
+            f"{os.path.join(debug_path, os.path.basename(self.clip_path))}.{int(frame_time*1000000)}.jpg",
            current_frame,
        )

@@ -290,7 +290,7 @@ def process(path, label, output, debug_path):
        1 for result in results if result[1]["true_positive_objects"] > 0
    )
    print(
-        f"Objects were detected in {positive_count}/{len(results)}({positive_count / len(results) * 100:.2f}%) clip(s)."
+        f"Objects were detected in {positive_count}/{len(results)}({positive_count/len(results)*100:.2f}%) clip(s)."
    )

    if output:
--- a/web/src/components/overlay/detail/SearchDetailDialog.tsx
+++ b/web/src/components/overlay/detail/SearchDetailDialog.tsx
@@ -543,16 +543,16 @@ function ObjectDetailsTab({
              )}
            </div>
          )}
-          {((config?.cameras[search.camera].genai.enabled && search.end_time) ||
-            !config?.cameras[search.camera].genai.enabled) && (
-            <Button
-              variant="select"
-              aria-label="Save"
-              onClick={updateDescription}
-            >
-              Save
-            </Button>
-          )}
+          {(config?.cameras[search.camera].genai.enabled && search.end_time) ||
+            (!config?.cameras[search.camera].genai.enabled && (
+              <Button
+                variant="select"
+                aria-label="Save"
+                onClick={updateDescription}
+              >
+                Save
+              </Button>
+            ))}
        </div>
      </div>
    </div>